nushell.github.io icon indicating copy to clipboard operation
nushell.github.io copied to clipboard

Update Cookbook direnv instructions for new std helper

Open NotTheDr01ds opened this issue 8 months ago • 3 comments

As mentioned in https://github.com/nushell/nushell/pull/15569 - This updates the direnv formula in the Cookbook to utilize the new std/config env-conversions helper.

NotTheDr01ds avatar Apr 14 '25 17:04 NotTheDr01ds

Nice!

What is to be done with this tip on the page?

you can follow the nu_scripts of Nushell for the always up-to-date version of the hook above

nu-scripts uses a different implementation. In fact, 2 days ago a separate fix was introduced for the nu-scripts implementation.

Currently it would be confusing for users whether to use the solution in the cookbook or the one in nu_scripts. Maybe it would be smart to keep only a single implementation around, considering that this is not the first time that the implementation in the cookbook and nu-scripts have diverged?

LemmusLemmus avatar Apr 14 '25 18:04 LemmusLemmus

Maybe it would be smart to keep only a single implementation around

That's my thought as well. Seems to be a better fit for the Cookbook, both in terms of discoverability and the fact that the nu_scripts is just a "snippet" more than anything.

NotTheDr01ds avatar Apr 14 '25 22:04 NotTheDr01ds

Put in https://github.com/nushell/nu_scripts/pull/1084 to remove the nu_scripts version. Also updated verbage.

NotTheDr01ds avatar Apr 15 '25 00:04 NotTheDr01ds

Can this also be merged now, since nushell/nu_scripts#1084 has been merged?

LemmusLemmus avatar Nov 27 '25 02:11 LemmusLemmus

It would be great if this could be merged. The tip that this removes contains a link to a file that was removed by the aforementioned PR and I was about to open a fix when I noticed that this change was pending.

cc the most recent commiters @cptpiepmatz @fdncred

ofek avatar Dec 03 '25 02:12 ofek

Thanks

fdncred avatar Dec 03 '25 11:12 fdncred