Review of "Applications Startup Time Measure" version 4

Details Page Preview

Shows startup time of an application

Extension Homepage
https://gitlab.gnome.org/3v1n0/gnome-shell-extension-apps-startup-measure

No comments.

Diff Against

Files

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

All Versions

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

Previous Reviews on this Version

JustPerfection active
1. Log should be only available in debug mode: https://gjs.guide/extensions/review-guidelines/review-guidelines.html#no-excessive-logging 2. It's better to temporary change the `HIDE_TIMEOUT` instead of monkey patching `timeout_add()`: https://gitlab.gnome.org/GNOME/gnome-shell/-/blob/633c8687806b2d1450146be09c439bd077e37a8c/js/ui/osdWindow.js#L10 3. Add Git URL to the metadata.json if the package has one: https://gjs.guide/extensions/review-guidelines/review-guidelines.html#metadata-json-must-be-well-formed If you need any help with your extension you can ask us on: - [GNOME Matrix Channel](https://matrix.to/#/#extensions:gnome.org) - IRC Bridge: irc://irc.gimpnet.org/shell-extensions
3v1n0 posted a review
1. I think it's not excessive, it will only print the message when app is launched, and the idea is to have such information for being parsed externally if needed. 2. Done in next version 3. As well.