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.

Sign up for GitHub

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

Jump to bottom

Update nixpkgs to track nixos-21.11 branch and update other inputs #11

Merged
merged 1 commit into from
Dec 3, 2021
Merged

Update nixpkgs to track nixos-21.11 branch and update other inputs #11

merged 1 commit into from
Dec 3, 2021

Conversation

montchr
Copy link
Contributor

@montchr montchr commented Dec 3, 2021

Fixes #5

cole-h
cole-h requested changes Dec 3, 2021
View reviewed changes
Copy link
Owner

@cole-h cole-h 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 the PR! Just one minor request:

flake.nix Outdated Show resolved Hide resolved
@montchr montchr changed the title Update nixpkgs to track release-21.11 branch and update other inputs Update nixpkgs to track nixos-21.11 branch and update other inputs Dec 3, 2021
cole-h
cole-h approved these changes Dec 3, 2021
View reviewed changes
Copy link
Owner

@cole-h cole-h left a comment

Choose a reason for hiding this comment

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

Diff LGTM, thanks!

I can't test this right now, so I'll merge it tomorrow (provided all goes well).

Also cc @enderger -- does this PR fix your issue from #5?

@cole-h
Copy link
Owner

cole-h commented Dec 3, 2021

Everything seems to still be working. Thanks!

@cole-h cole-h merged commit 0bfd2f3 into cole-h:main Dec 3, 2021
montchr added a commit to montchr/dotfield that referenced this pull request Feb 21, 2022
@montchr
chore(inputs|secrets): track orig nix-darwin/agenix-cli + agenix fork
d5f10f7 
see ryantm/agenix#79

- ryantm/agenix -> montchr/agenix -- includes changes from the above PR
for nix-darwin support

- montchr/nix-darwin -> LnL7/nix-darwin -- since the above PR on
ryantm/agenix itself does the job, no need to rely on a nix-darwin PR
that may never be merged...

- montchr/agenix-cli -> cole-h/agenix-cli -- i never updated since
cole-h/agenix-cli#11 was merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cole-h cole-h cole-h approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Build fails under latest unstable-small branch
2 participants
@montchr @cole-h