Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade nix to 0.28 #5

Merged
merged 3 commits into from
Mar 1, 2024
Merged

Conversation

sunshowers
Copy link
Contributor

Thanks for maintaining this crate! Sending a PR to update nix to the latest version, so it picks up (in particular) nix-rust/nix#2275.

Thanks!

Copy link
Owner

@andreacorbellini andreacorbellini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for submitting this! It looks like it's failing to build on Linux due to some missing Option, could you please fix that?

@sunshowers
Copy link
Contributor Author

Whoops, haha -- should have tested there! I usually do my dev on Linux, but was on illumos while debugging this.

@andreacorbellini
Copy link
Owner

Sorry, looks like the CI is failing due to linting. If you run cargo fmt it should fix the problem. Thanks!

@sunshowers
Copy link
Contributor Author

My bad! I was editing using raw vim rather than vscode for $reasons.

Would you be willing to change the "require approval" setting to make it so that it only applies to users not new to GitHub?

image

@andreacorbellini andreacorbellini merged commit bc8c638 into andreacorbellini:master Mar 1, 2024
6 checks passed
@sunshowers sunshowers deleted the nix branch March 1, 2024 19:07
@sunshowers
Copy link
Contributor Author

Thank you! Wondering if you can get a release out soon.

@andreacorbellini
Copy link
Owner

andreacorbellini commented Mar 1, 2024 via email

@andreacorbellini
Copy link
Owner

Version 0.1.3 is now published with your changes: https://crates.io/crates/atomic-write-file/0.1.3. Enjoy, and thanks again for your contribution!

@sunshowers
Copy link
Contributor Author

Appreciate it! Thanks again for being so responsive :)

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.

None yet

2 participants