Skip to content

syncthing-status@geo-sudo: Initial Release#1825

Open
Geo-sudo wants to merge 2 commits intolinuxmint:masterfrom
Geo-sudo:master
Open

syncthing-status@geo-sudo: Initial Release#1825
Geo-sudo wants to merge 2 commits intolinuxmint:masterfrom
Geo-sudo:master

Conversation

@Geo-sudo
Copy link
Copy Markdown

A simple desklet to monitor Syncthing folders.

screenshot

@github-actions
Copy link
Copy Markdown

Best-practices scanner

This is a regex-based check for API usage that can pose security, performance or
maintainability issues, or that may already be provided by Cinnamon. Having code flagged
by it doesn't automatically disqualify a pull request.

This check is not perfect will not replace a normal review.


Found 1 potential issue(s):

⚠️ hardcoded_data_dir

syncthing-status@geo-sudo/files/syncthing-status@geo-sudo/desklet.js:9

Gettext.bindtextdomain(UUID, imports.gi.GLib.get_home_dir() + "/.local/share/locale");

Avoid hardcoding .local/share in paths. Use GLib.get_user_data_dir() instead,
which respects the XDG_DATA_HOME environment variable.


Automated pattern check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant