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

Non-content edits to pills 9-14 #195

Merged
merged 8 commits into from
Feb 9, 2024
Merged

Conversation

peter-mlabs
Copy link

This PR (and future PRs I'll be submitting in the same vein) is focused on improving grammar, voicing, stylistic consistency, and readability. As a relatively inexperienced Nix user, I don't have much to add in terms of content or technical commentary; I'd simply like to improve the quality of the existing documentation.

I've found that the earlier pills have a slightly different tone than the later pills -- some of which almost seems like it can be attributed to different authors -- and that the writing style becomes weaker as the material grows in complexity. As such, I only committed to undertaking this editing starting at pill #9 (the point at which I felt my own comprehension beginning to suffer), and I plan on editing as I go through the remaining pills. Once I'm finished, I'll try to go back and revise from the beginning.

@peter-mlabs peter-mlabs changed the title Non-content edits to pills 9-12 Non-content edits to pills 9-13 Jun 26, 2022
@peter-mlabs peter-mlabs changed the title Non-content edits to pills 9-13 Non-content edits to pills 9-14 Jun 26, 2022
@drupol
Copy link
Contributor

drupol commented Oct 18, 2023

What's the status of this PR ? I personally like the changes.

@fricklerhandwerk
Copy link
Contributor

Only glanced over it, but looks fine to merge if conflicts are resolved. (My mobile view thinks there are some left, maybe it's wrong.)

@fricklerhandwerk
Copy link
Contributor

fricklerhandwerk commented Feb 9, 2024

Thanks @henrik-ch for keeping things moving!

@fricklerhandwerk fricklerhandwerk merged commit a66fd07 into NixOS:master Feb 9, 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
None yet
Development

Successfully merging this pull request may close these issues.

4 participants