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

Improve NixOS/Wayland documentation #3614

Closed

Conversation

Munksgaard
Copy link

Objective

Some additional requirements for Wayland on NixOS are not documented. Also, there are some unnecessary package qualifications in the current documentation.

Solution

Remove unnecessary package qualifications and document Wayland requirements on NixOS.

Since the code stump already includes `with pkgs;` at the top, we do not need to
further qualify these imports.
@github-actions github-actions bot added the S-Needs-Triage This issue needs to be labelled label Jan 9, 2022
@alice-i-cecile alice-i-cecile added C-Docs An addition or correction to our documentation O-Linux Specific to the Linux desktop operating system and removed S-Needs-Triage This issue needs to be labelled labels Jan 9, 2022
@alice-i-cecile
Copy link
Member

Thanks for the contribution! It looks good at first pass, but I'm not familiar with NixOS. I'll run this by some existing users :)

@mockersf
Copy link
Member

mockersf commented Jan 9, 2022

There is already #3527 that is trying to improve nixos documentation. Would this pr conflicts with it?

@Munksgaard
Copy link
Author

Oh, that's right! I think our efforts are probably better spent on that PR instead :)

@Munksgaard Munksgaard closed this Jan 9, 2022
@mockersf
Copy link
Member

mockersf commented Jan 9, 2022

thanks for your help!

@Munksgaard Munksgaard deleted the improve-wayland-documentation branch January 9, 2022 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Docs An addition or correction to our documentation O-Linux Specific to the Linux desktop operating system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants