Skip to content

nix: add home-manager module#2

Open
NotAShelf wants to merge 1 commit intoquickshell-mirror:masterfrom
NotAShelf:hm-module
Open

nix: add home-manager module#2
NotAShelf wants to merge 1 commit intoquickshell-mirror:masterfrom
NotAShelf:hm-module

Conversation

@NotAShelf
Copy link

  • Formats the flake with alejandra (nixfmt users can fuck a hat)
  • Adds a home-manager module

@NotAShelf
Copy link
Author

mr fox what are you waiting for

@outfoxxed outfoxxed force-pushed the master branch 2 times, most recently from b9c5698 to d1a1727 Compare February 21, 2025 10:38
Copy link

@cardboardcpu cardboardcpu left a comment

Choose a reason for hiding this comment

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

Annoying, pedantic nits.

systemd.user.services.quickshell = {
Install.WantedBy = [cfg.systemdTarget];

Unit = {

Choose a reason for hiding this comment

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

Pedantic nit.: Providing a link to the docs here would be nice.

];
};

Service = {

Choose a reason for hiding this comment

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

nit: What about adding some hardening options by default? Just as a good praxis and to get more smileys from SystemD. :D

@Actro25 Actro25 mentioned this pull request Sep 18, 2025
@outfoxxed outfoxxed force-pushed the master branch 2 times, most recently from e170d91 to 1d94144 Compare September 29, 2025 06:56
@y-oun-g y-oun-g mentioned this pull request Dec 8, 2025
@RobinD2001 RobinD2001 mentioned this pull request Dec 16, 2025
@Effectforward Effectforward mentioned this pull request Feb 14, 2026
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