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

Update 09-automatic-runtime-dependencies.md - fix NixOps link #238

Merged
merged 1 commit into from
Apr 12, 2024

Conversation

noamraph
Copy link
Contributor

The link to NixOps was corrupted by mistake in this PR: https://github.com/NixOS/nix-pills/pull/181/files#diff-a53cbc0f08ac225fd5073d54147a9a6ba2ebbdbd0e5164711ab362d0439cc27aR265

It was changed by mistake from http://nixos.org/nixops/manual/#chap-introduction to https://nixos.org/manual/nix/stable/introduction.html.

I think that at least for now, it would be better to link to https://github.com/NixOS/nixops, as the old link just links to an obscure in-development HTML document, and the github link starts with a warning "NixOps is in low-maintenance mode and probably not suited for new projects. Use at your own risks", and also links to an experimental NixOps4.

The link to NixOps was corrupted by mistake in this PR: https://github.com/NixOS/nix-pills/pull/181/files#diff-a53cbc0f08ac225fd5073d54147a9a6ba2ebbdbd0e5164711ab362d0439cc27aR265

It was changed from http://nixos.org/nixops/manual/#chap-introduction to https://nixos.org/manual/nix/stable/introduction.html.

I think that at least for now, it would be better to link to https://github.com/NixOS/nixops, as the old link just links to an obscure in-development HTML document, and the github link starts with a warning "NixOps is in low-maintenance mode and probably not suited for new projects. Use at your own risks", and also links to an experimental NixOps4.
@drupol drupol merged commit 514b3fe into NixOS:master Apr 12, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants