Review of "Adjust Display Brightness 2" version 2

Details Page Preview

Control external monitor brightness from GNOME Shell using DDC/CI. Requires ddcutil and permission for the logged-in user to access the display's I2C device.

Extension Homepage
https://gitlab.com/bruno.englert/gnome-extension-brightness

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)

Shexli did not find any issues.

All Versions

Version Status
3 Unreviewed
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
This package seems to be generated by AI. Please note, we have a rule for that: [EGO Review Guidelines: AI](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#extensions-must-not-be-ai-generated)
JustPerfection posted a review
If you need any help with your extension you can ask us on: - [GNOME Extensions Matrix Channel](https://matrix.to/#/#extensions:gnome.org) - IRC Bridge: irc://irc.gimpnet.org/shell-extensions
brunoenglert posted a review
This is the updated version of https://extensions.gnome.org/extension/4652/adjust-display-brightness/ (was also developed by me), so settings can be changed, rgb color and contrast can be changed, and other extra stuff.
JustPerfection posted a review
Well, this code still has unnecessary lines that seem like AI generated code. If you want it reviewed, you should follow our guidelines: https://gjs.guide/extensions/review-guidelines/best-practices.html
brunoenglert posted a review
I did follow https://gjs.guide/extensions/review-guidelines/best-practices.html . Can you give an example of an "unnecessary line"?
JustPerfection posted a review
Please search the entire package for: - https://gjs.guide/extensions/review-guidelines/best-practices.html#avoid-unnecessary-try-catch-wrappers - https://gjs.guide/extensions/review-guidelines/best-practices.html#avoid-unnecessary-checks