nufmt icon indicating copy to clipboard operation
nufmt copied to clipboard

Move Nix-related files to nushell/integrations

Open kubouch opened this issue 2 years ago • 5 comments

Is your feature request related to a problem? Please describe.

We recently set up a community maintained https://github.com/nushell/integrations to keep package manager and other integration files for Nushell projects there. The intent is to avoid cluttering our repositories with 3rd party tool files that the core team has to maintain.

Describe the solution you'd like

Move Nix-related files to https://github.com/nushell/integrations

Describe alternatives you've considered

Since the core team doesn't actively develop nufmt (yet?), it depends on the active nufmt developers if they want these in and maintain them. But once we scale up the development, we might move them, since that's the direction we've followed in Nushell itself and recently nupm.

kubouch avatar Jan 28 '24 11:01 kubouch

sounds very nice to have everything centralized into once place

amtoine avatar Jan 28 '24 11:01 amtoine

i don't know if simply moving flake.nix and flake.lock to a new nufmt/ directory in nushell/integrations would be enough?

amtoine avatar Jan 28 '24 11:01 amtoine

I'm thinking a better idea would be to have a directory per package manager. I imagine package managers could reuse some files between projects, or have a top-level Nushell package with nupm/nufmt/... as its sub-packages. So it could look something like

nushell/integrations
+-- apt/
      +-- nufmt/
      +-- nupm/
      +-- nushell/
      +-- common-stuff/
+-- nix/
      +-- nufmt/
      +-- nupm/
      +-- nushell/
      +-- common-stuff/

Note that just copy-pasting the current flake files (the .envrc has some nix stuff in it as well) to nushell/integrations won't work, the flake would need to change to fetch nufmt from GitHub instead of building from local file tree.

kubouch avatar Jan 28 '24 11:01 kubouch

yup, all of this makes perfect sense :ok_hand:

amtoine avatar Jan 28 '24 15:01 amtoine

Yes! I completely agree!

AucaCoyan avatar Jan 28 '24 15:01 AucaCoyan