Review "Simple monitor" version 1 - GNOME Shell Extensions

Review of "Simple monitor" version 1

Details Page Preview

Simple monitor

by fcaballero

A simple panel button that shows CPU and RAM memory usage. Clicking shows the top 10 processes using CPU and memory.

Extension Homepage: https://github.com/fcaballerop/simple-monitor-gnome-shell-extension

No comments.

FAQ

You can upload the next version by the same form you've posted this version.
You can change the extension icon by clicking on the icon in your extension page.
In case you want to inactivate an approved version, click on inactivate link in your extension page.
You can find developer guides in gjs.guide.
If you need any help with your extension, you can ask us on GNOME Matrix Channel.

Files

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

All Versions

Version	Status
8	Active
7	Active
6	Active
5	Active
4	Active
3	Active
2	Rejected
1	Active

Previous Reviews on this Version

andyholmes active December 5, 2020

You should chain up at the end of your `_onDestroy()` override by calling `super._onDestroy();` otherwise your extension will leak memory. Additionally, it is generally a bad idea to spawn subprocesses syncrhonously, especially in a loop. Your extension will block the main loop of gnome-shell for a significant period every 5 seconds.