You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hey! Nice project, but there seems to be a few things that could see some love from the maintainer(s).
The usage of with lib;
Most good nix projects actively avoid with scopes. For better maintainability, those should be replaced with explicit inherits.
Lack of modularity
If this is meant as your personal config, and that only - it's perfectly fine. Otherwise, it will come off as too opinionated and could probably benefit from 1. an internal module system and 2. exposing some of the "pwning" related options as nixosModules that are directly importable from outside.
Confusing structure
Sticking everything in a modules directory is probably a bad idea. Maybe divide things by function so that it's easier to find what you're looking for?
The text was updated successfully, but these errors were encountered:
First of all, thank you very much for the comment and for the PR, it helps me a lot to improve this project.
This project started as a personal configuration of my work environment for pentesting and development of hacking tools among others. But it is true that I would like to set up something like what you indicate, a flake focused on facilitating the adoption of hacking tools and configurations for NixOS users.
I will try to follow your suggestions to improve the structure and especially the modules part. The ideal for me would be to keep this project as a system configuration so that users who are developers or pentesters give nixos a chance and at the same time other users who already have their own configuration can import the modules they need or want to test. I don't know if this is possible or not, I need to investigate it yet.
Hey! Nice project, but there seems to be a few things that could see some love from the maintainer(s).
with lib;
Most good nix projects actively avoid with scopes. For better maintainability, those should be replaced with explicit inherits.
If this is meant as your personal config, and that only - it's perfectly fine. Otherwise, it will come off as too opinionated and could probably benefit from 1. an internal module system and 2. exposing some of the "pwning" related options as nixosModules that are directly importable from outside.
Sticking everything in a modules directory is probably a bad idea. Maybe divide things by function so that it's easier to find what you're looking for?
The text was updated successfully, but these errors were encountered: