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

fix link to platforms.nix #31217

Merged
merged 1 commit into from Nov 4, 2017
Merged

fix link to platforms.nix #31217

merged 1 commit into from Nov 4, 2017

Conversation

BlessJah
Copy link
Contributor

@BlessJah BlessJah commented Nov 4, 2017

Motivation for this change

Fix broken link in documentation. I am not 100% sure if the file was moved or it's purpose changed.

I hope the issues will be fixed faster if I create PR than if I open issue.

@BlessJah
Copy link
Contributor Author

BlessJah commented Nov 4, 2017

@c0bw3b could you please check if the link points to correct document? Thanks!

@c0bw3b
Copy link
Contributor

c0bw3b commented Nov 4, 2017

Yes the link to platforms.nix is good.

But actually that file is not really helpful if one wants to know what platforms attributes are available. I find doubles.nix to be much more explicit and comprehensive.
Maybe the doc should refer to this file instead?

@grahamc
Copy link
Member

grahamc commented Nov 4, 2017

A nice improvement, doubles might be better. If there is a PR for that change, it'll likely merge too :)

@grahamc grahamc merged commit c371b83 into NixOS:release-17.09 Nov 4, 2017
@BlessJah BlessJah deleted the fix-link branch November 4, 2017 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants