feat(step): update KDE Plasmoids using libplasmoid-updater#1804
Open
uwuclxdy wants to merge 1 commit intotopgrade-rs:mainfrom
Open
feat(step): update KDE Plasmoids using libplasmoid-updater#1804uwuclxdy wants to merge 1 commit intotopgrade-rs:mainfrom
uwuclxdy wants to merge 1 commit intotopgrade-rs:mainfrom
Conversation
uwuclxdy
commented
Mar 1, 2026
Member
|
FYI the MSRV check in CI will fail because of the current MSRV of 1.84 which will be bumped to 1.88 in v17 (which I will release soon, I hope). You don't have to change the edition and you can ignore the MSRV check CI for now. |
uwuclxdy
force-pushed
the
plasmoid-updater
branch
from
2f63e50 to
fd4ad80
Compare
Contributor
Author
|
@GideonBear yeah, i fixed the ordering problem tho. they were ordered but not in the right place in proportion to the entire enum |
Contributor
Author
GideonBear
requested changes
Mar 2, 2026
Member
GideonBear
left a comment
GideonBear
left a comment
There was a problem hiding this comment.
Mostly good, I'll need to review plasmoid-updater of course but the API looks nice!
GideonBear
changed the title
GideonBear
requested changes
Mar 2, 2026
uwuclxdy
force-pushed
the
plasmoid-updater
branch
from
71d190c to
3db33d7
Compare
uwuclxdy
force-pushed
the
plasmoid-updater
branch
from
3db33d7 to
dd97361
Compare
barkleesanders
added a commit
to barkleesanders/topgrade
that referenced
this pull request
Mar 8, 2026
From PR topgrade-rs#1804 by uwuclxdy. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> # Conflicts: # Cargo.lock
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do
Adds integration with
libplasmoid-updaterto update KDE Plasma components (plasmoids) user and system-wide.Fixes #1020
It can also be used as a standalone program for those that wish to test (or just use) the library itself: https://github.com/uwuclxdy/plasmoid-updater
Standards checklist
CONTRIBUTING.mdFor new steps
--dry-runoption works with this step--yesoption works with this step if it is supported bythe underlying command
My best try at this ig. Needs testing (edge cases, error handling). As far as library is concerned, the code is a messy and worse I'd like it to be, but it works! It successfully finds and updates components that either Discover or Apdatify sometimes doesn't find.