Review of "CORE STATS" version 11

Details Page Preview

Display hardware temperatures (CPU, GPU, NVMe, etc.)

Extension Homepage
https://github.com/devilbd/gnome-extensions/tree/main/core-stats

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) warning 1

Shexli found 1 issue that may need reviewer attention.

EGO-L-002 warning

objects created by extension should be destroyed in disable()

Objects assigned in `enable()` are missing matching `.destroy()` calls in `disable()` or its helper methods.

Destroy all objects

  • extension.js:480
            this._coreWidget = new St.BoxLayout({
                vertical: true,
                style_class: 'core-stats-card',
                reactive: true,
                can_focus: false,
                track_hover: false,
                x_expand: true,
                y_expand: true
            })
  • extension.js:537
            this._mbWidget = new St.BoxLayout({
                vertical: true,
                style_class: 'core-stats-card core-stats-mb-card',
                reactive: true,
                can_focus: false,
                track_hover: false,
                x_expand: true,
                y_expand: false
            })

All Versions

Version Status
12 Unreviewed
11 Rejected
10 Rejected
9 Rejected
8 Rejected
7 Rejected
6 Rejected
5 Rejected
4 Rejected
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

devilbd auto- rejected
Auto-rejected because of new version 12 was uploaded