dashy-ui: 3.3.1 -> 4.0.5#513946
Merged
Merged
Conversation
Contributor
|
After reading some of the release discussions I think it's best to wait for some more hotfix releases before putting this through, to allow time for some bugs to be ironed out. I'll test and merge #512221 now and keep an eye on the releases. |
Contributor
Author
|
Updated to 4.0.5 |
Contributor
Contributor
Author
TheRealGramdalf
approved these changes
May 5, 2026
TheRealGramdalf
left a comment
Contributor
There was a problem hiding this comment.
I don't think I could add much either, as my config consists of similar content. I'm happy enough with that, I mostly wanted to double check that the results from the nixosTests weren't passing due to a logic error or similar. Should be ready to merge.
Member
|
As the author described, not a breaking change for end users. |
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
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.


This new major release should be mostly about internal rewrites to Vue 3 and an update to Node 24. The config is supposed to be backward compatible. See: https://github.com/Lissy93/dashy/releases/tag/4.0.0.
We already have #512221 open, but since this is a major update, should we supersede it, or keep them separate?
I also tested the change on my real config, and it worked from the first try. Although I only have sections and links, I don't use any widgets.
Things done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.