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

let installer blurb point to community page #7419

Merged
merged 1 commit into from
Dec 7, 2022

Conversation

fricklerhandwerk
Copy link
Contributor

being too specific about it requires more maintenance (or otherwise
produced more confusion and churn), since these points of contact change
over time.

@fricklerhandwerk fricklerhandwerk changed the title point installer blurb to community page let installer blurb to to community page Dec 7, 2022
being too specific about it requires more maintenance (or otherwise
produced more confusion and churn), since these points of contact change
over time.
@thufschmitt
Copy link
Member

I don't have a strong opinion on that. I'll just point out that https://nixos.org/community/ points to a different Matrix channel than the one in the current message (#community:nixos.org versus the more specifix #nix:nixos.org)

@fricklerhandwerk
Copy link
Contributor Author

I don't think we want everyone who the installer fails on to post to the Nix channel. The purpose of that message should be to direct beginners to the established support paths which are more accessible and easier to maintain than this obscure piece of code in the installer.

@edolstra edolstra merged commit 9c97685 into NixOS:master Dec 7, 2022
@fricklerhandwerk fricklerhandwerk changed the title let installer blurb to to community page let installer blurb point to community page Dec 7, 2022
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/tweag-nix-dev-update-41/23848/1

@fricklerhandwerk fricklerhandwerk deleted the installer-contact branch January 5, 2023 14:46
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.

4 participants