astro / deadnix

Scan Nix files for dead code
GNU General Public License v3.0
447 stars 15 forks source link

Merge with statix #36

Closed hab25 closed 1 year ago

hab25 commented 1 year ago

This will likely help increase the discoverability and lower the barrier to entry of using both tools.

see also https://github.com/nerdypepper/statix/issues/41

astro commented 1 year ago

No serious objection but I would like to point out that the tools perform two very different things:

hab25 commented 1 year ago

I don't understand that difference well enough, but it seems to me like something that is primarily relevant for maintainers but not so much to an end-user.

astro commented 1 year ago

Even end-users care about the UNIX Philosophy™:

Make each program do one thing well. To do a new job, build afresh rather than complicate old programs by adding new "features".

hab25 commented 1 year ago

Good point. I also approve of the UNIX philosophy, and in accordance with it I think a new metalinter project is better than merging. I say "new" only because I don't know of any existing ones.

Feel free to close this issue if you want.