update flake and rust dependencies #317
Open
Conversation
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
|
The only skipped dependency is |
This was referenced Mar 27, 2025
|
Tested the PR locally, works fine. No regression appears to be present. |
This a naive update with minor adjustements for new APIs. Most changes are clap related because their macro naming has changed.
Extend ast handling for nodename with dots, see serokell#307
|
After an hour on a builder, nixos vm test failed, seems like activation CLI needs some more work: |
weriomat
reviewed
May 13, 2025
There was a problem hiding this comment.
Hey, I have debugged the issue with the tests.
Since you bumped the nixpkgs version, this commit removes the switch-to-configuration script by default.
We should enable this option in the common config because the test VMs need to have the new rust script in path (since they cannot fetch it while being run)
weriomat
suggested changes
May 13, 2025
…loys [This](NixOS/nixpkgs@4cfbbb3) commit has disabled the option by default, but we need the `switch-to-configuration` binary in order to successfully deploy to the server
Fix tests + overlays output
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 is a naive update with minor adjustments for new APIs. Most changes are clap related because their macro naming has changed.