Skip to content

Nushell support for Global packages - #2743

Merged
Lagoja merged 9 commits into
jetify-com:mainfrom
omerxx:main
Feb 13, 2026
Merged

Nushell support for Global packages #2743
Lagoja merged 9 commits into
jetify-com:mainfrom
omerxx:main

Conversation

@omerxx

Copy link
Copy Markdown
Contributor

Summary

To load global packages with Nushell the user is required to eval $(devbox global shellenv). Nushell doesn't have an eval equivalent and we need to:

  1. Convert the output
  2. Do this on the fly when the environment is loaded
  3. Maintain previous behavior

How was it tested?

Locally with Nushell based setup trying to load both bash and nu scripts.

Community Contribution License

All community contributions in this pull request are licensed to the project
maintainers under the terms of the
Apache 2 License.

By creating this pull request, I represent that I have the right to license the
contributions to the project maintainers under the Apache 2 License as stated in
the
Community Contribution License.

@omerxx

Copy link
Copy Markdown
ContributorAuthor

Hopefully this solves #2318

@NeckBeardPrince

Copy link
Copy Markdown

One day

@Lagoja

Copy link
Copy Markdown
Contributor

I tested this out and it seems to work on my machine, but I'm a little confused why this didn't trigger out CI/CD checks?

@LagojaLagoja closed this Feb 5, 2026
@LagojaLagoja reopened this Feb 5, 2026
- Remove unused refreshCmdForShell function (lint error)
- Update process-compose version in flakeref test from v0.40.2 to v1.46.0
(old version used Go 1.19.3 which fails on newer macOS)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@RWayne93

Copy link
Copy Markdown

surprised this is still open.

Signed-off-by: John Lago <750845+Lagoja@users.noreply.github.com>
Signed-off-by: John Lago <750845+Lagoja@users.noreply.github.com>
@Lagoja

Copy link
Copy Markdown
Contributor

surprised this is still open.
Going to put this in the next dev release, had some issues testing and running the workflows that now see to be resolved

@Lagoja
Lagoja merged commit 0b9e37d into jetify-com:mainFeb 13, 2026
24 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@omerxx@NeckBeardPrince@Lagoja@RWayne93