GitHub - FullOvellas/nix_config
github.com
Contribute to FullOvellas/nix_config development by creating an account on GitHub.

I think it’s about time I improved a bit on my Nix and NixOS knowledge, and I guess the best way is having someone more experienced critique my monstrosity of a setup.

Maybe it’s too modular? Not modular enough? Convoluted? Complete garbage altogether?

Go to the repo and find out! Just be sure to let me know after.

    • fullovellas@lemm.eeOPEnglish
      3·
      1 day ago

      I read your issue and you left some great advice. Thank you for taking the time to write something thoughtful! As you probably guessed I really had barely any idea of how the module system worked, but it’s nice seeing it’s a bit similar to Rust’s. I will look into the switch script, which will be convenient, and apply your suggestions, since I basically agree with all of them 🙌

    • jevans ⁂@lemmy.mlEnglish
      2·
      1 day ago

      recursiveMerge

      Can you explain how you would use this here? Would this make files like this in my config unnecessary?

      • lelgenio@lemmy.mlEnglish
        2·
        1 day ago

        I wasn’t very clear in how I wrote those points, I think you got it backwards.

        My points was that OP was using recursiveMerge, and I think that is not a very common or useful pattern to follow, because you evade the module system, for no good reason(?).

        I think you are already following the most common pattern out there. If you would like to, there are ways to avoid having a default.nix file for each module, but in my opinion that only makes it harder for other people to understand your modules.