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

feat: fixed documentation #497

Closed
wants to merge 0 commits into from

Conversation

DockterTeagle
Copy link
Contributor

The documentation had a deprecated argument in the home-manager example by using configuration, this aims to fix that by providing an example of how to fix it in the current nixos unstable branch(which at the time of writting is 24.05), through a comment.

README.md Outdated
@@ -29,6 +29,8 @@ If you are using flakes to configure your system, add to your nixpkgs overlays a
{
nixpkgs.overlays = overlays;
};
#replace the configuration code segment above with the following if running nixos unstable:
Copy link
Member

Choose a reason for hiding this comment

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

Since 22.11 this is the way to do it, you should just replace the entire configuration part, replace it with modules and remove the comment.

@DockterTeagle
Copy link
Contributor Author

sorry I have not been the best on responding, I have finals. Thought I should let you know why I was not working on this

@DockterTeagle
Copy link
Contributor Author

I have done the requested changes locally, how can I show those changes here in this pull request?

@Kranzes
Copy link
Member

Kranzes commented May 8, 2024

Commit and push?

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