direnv: hopefully final nushell fix

[K900]

Description

I hate this.

Checklist

• Change is backwards compatible.

• Code formatted with ./format.

• Code tested through nix-shell --pure tests -A run.all or nix develop --ignore-environment .#all using Flakes.

• Test cases updated/added. See example.

• Commit messages are formatted like

  {component}: {description}
  
  {long description}
  

  See CONTRIBUTING for more information and recent commit messages for examples.

• If this PR adds a new module

  • Added myself as module maintainer. See example.

Maintainer CC

[rycee]

I fixed the test case and merged to master now. Hope it works better now! Have to go sleep so will check back tomorrow.

[nyabinary]

This breaks nushell as it overrides the default env.nu breaking the behavior of nushell in general isn't there a way to just append the direnv stuff instead of overwriting the entire env.nu file?

[K900]

No it doesn't? That's why the mkAfter is there.

[nyabinary]

No it doesn't? That's why the mkAfter is there.

#5527
nushell/nushell#13174

Does for me I just want to add a alias and stuff in my config and it overrides the entire default env.nu already in place instead of appending to it.

[K900]

That's unrelated to this change though?

[nyabinary]

That's unrelated to this change though?

But the direnv thing is broken in general because if you enable the direnv support for nushell it currently breaks nushell:

Pops up a nu::shell::non_zero_exit_code when normally its not supposed to do that :P

[K900]

That has nothing to do with direnv, and it is supposed to do that starting with Nushell 0.98 (see the original chain of PRs)