Skip to content

Nushell support for Global packages #2743

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

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

Conversation

@omerxx
Copy link
Contributor

@omerxx omerxx commented Nov 9, 2025

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
Contributor Author

omerxx commented Nov 9, 2025

Hopefully this solves #2318

@NeckBeardPrince
Copy link

One day

@Lagoja
Copy link
Collaborator

Lagoja commented Feb 3, 2026

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?

@Lagoja Lagoja closed this Feb 5, 2026
@Lagoja Lagoja 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

RWayne93 commented Feb 8, 2026

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
Collaborator

Lagoja commented Feb 13, 2026

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:main Feb 13, 2026
24 checks passed
Sign up for free to 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