Skip to content

speedpup@taskpuppynatani: add SpeedPup network desklet - #1888

Open
TaskPuppyNatani wants to merge 2 commits into
linuxmint:masterfrom
TaskPuppyNatani:speedpup-submission
Open

speedpup@taskpuppynatani: add SpeedPup network desklet#1888
TaskPuppyNatani wants to merge 2 commits into
linuxmint:masterfrom
TaskPuppyNatani:speedpup-submission

Conversation

@TaskPuppyNatani

Copy link
Copy Markdown

Summary

Adds SpeedPup, a Cinnamon desklet for live network monitoring and on-demand internet speed tests.

Features include:

  • Live download and upload throughput
  • Live traffic graph
  • Automatic and manual network interface selection
  • Adjustable desklet size
  • Configurable graph history and height
  • On-demand download, upload, and ping tests
  • Persistent last speed-test result and timestamp
  • Self-contained Python speed-test helper using the LibreSpeed server network

Validation

  • ./validate-spice speedpup@taskpuppynatani passes with no errors
  • ./test-spice speedpup@taskpuppynatani succeeds

No additional speed-test package is required.

@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. Most findings
are advisory and do not automatically disqualify a pull request.

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


Found 1 potential issue(s):

⚠️ WARNING

⚠️ hardcoded_data_dir

speedpup@taskpuppynatani/files/speedpup@taskpuppynatani/desklet.js:24

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.

@TaskPuppyNatani

Copy link
Copy Markdown
Author

Hi! Just checking in on this submission. I’ve addressed the automated hardcoded_data_dir warning, and the updated pattern check is passing. Please let me know if there’s anything else I should change or provide for review. Thanks for your time!

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