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

quickstart fixes #236

Merged
merged 5 commits into from
Oct 20, 2023
Merged

quickstart fixes #236

merged 5 commits into from
Oct 20, 2023

Conversation

phaer
Copy link
Member

@phaer phaer commented Oct 19, 2023

trying to fix formatting issues and clarify it a bit, incorporating feedback by @chaoflow at oceansprint

@chaoflow
Copy link

@phaer https://github.com/nix-community/nixos-anywhere/blob/main/docs/quickstart.md#steps-required-to-run-nixos-anywhere --- Step 2, nix flake init seems to not be needed, I suggest to instead explain to copy all files from the example repo, or use flake template as discussed today.

cc @Mic92

...as we've received feedback that this is  more
confusing for newer users than starting from the
example and it's probably safe to assume more experienced
users will be able to figure it out on their own.
@phaer
Copy link
Member Author

phaer commented Oct 20, 2023

@chaoflow Ah right, I was a bit lazy yesterday and sleep-deprived enough that I wasn't sure whether my changes would be a net-positive in terms of readability. Still not totally recovered, but that might be good enough?

@Mic92
Copy link
Member

Mic92 commented Oct 20, 2023

Did wanted to add how to download the configuration generated by nixos-generate-config?

@phaer
Copy link
Member Author

phaer commented Oct 20, 2023

Didn't really get what you meant here: an scp command? L

@@ -70,11 +63,12 @@ below.
```

This creates the necessary configuration files under `/mnt/etc/nixos/`, which
you can then customize as needed.
you can then customize as needed and copy them to your local machine in order
Copy link
Member

Choose a reason for hiding this comment

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

Ah, you kind of mentioning this here. I missed that sentence on my phone.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, just saw that I somehow posted a butchered response from my phone, hope that wasn't confusing

@Mic92
Copy link
Member

Mic92 commented Oct 20, 2023

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Oct 20, 2023

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at c1d2893

@mergify mergify bot merged commit c1d2893 into main Oct 20, 2023
3 checks passed
@mergify mergify bot deleted the quickstart branch October 20, 2023 15:09
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.

3 participants