numtide / devshell

Per project developer environments
https://numtide.github.io/devshell/
MIT License
1.22k stars 87 forks source link

imp: satisfy a top level untyped output scheme #195

Closed blaggacao closed 2 years ago

blaggacao commented 2 years ago

when deSystemized allows for immediate usage of devshell:

blaggacao commented 2 years ago

It's just vanity: (and boldness :smile:)

  l.mapAttrs (_: devshell.legacyPackages.mkShell) {
    default = { extraModulesPath, pkgs, ... }: { /* ... */ };
  }

-->

  l.mapAttrs (_: devshell.mkShell) {
    default = { extraModulesPath, pkgs, ... }: { /* ... */ };
  }
zimbatm commented 2 years ago

Isn't this going to pollute the nix flake check output?

blaggacao commented 2 years ago

Oh absolutely. It upfront challenges the fixed schema authority of the nix CLI, alltogether.

blaggacao commented 2 years ago

The current flake schema is use-case skewed and is a hinderance in unleaching the full potential of nix flakes.

blaggacao commented 2 years ago

Think about the immense amount of context tucked away behind the legacyPackages identifier. :smile:

zimbatm commented 2 years ago

Thanks for clarifying. I agree that the current flake output schema is missing a bunch of attributes, but not that we should therefore abandon all classification like that.