Review of "System HUD" version 28

Details Page Preview

Clean UI system monitor for CPU, GPU, memory, storage, network, power.

Extension Homepage
https://github.com/LalaloyXyz/System-info-HUD

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

Shexli (experimental) error 1 manual_review 1

Shexli found 2 issues that may need reviewer attention.

EGO-A-005 manual_review

extensions should not access the clipboard directly

Direct clipboard access via `St.Clipboard.get_default()` requires reviewer scrutiny.

Review Guidelines

  • uiManager.js:1118
    St.Clipboard.get_default()

EGO-M-004 error

metadata shell-version must only include plausible stable releases and at most one development release

Field `shell-version` contains invalid values, more than one development release, or implausible future releases.

metadata.json must be well-formed

  • metadata.json
    shell-version=['45', '46', '47', '48', '49', '50', '51']

All Versions

Previous Reviews on this Version

JustPerfection rejected
1. Timeout should be removed on destroy and before creating a new one: - line 1357 `uiManager.js` - line 70 `modules/baseModule.js` [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 2. You cannot do this much spawn commands in shell process. Move everything out to an app and make that app a dependency of this extension. They can communicate with d-bus: [D-Bus Guide](https://gjs.guide/guides/gio/dbus.html) Please avoid sending large diff. It can wait even more than a month since the review queue is very busy.