- Notifications
You must be signed in to change notification settings - Fork373
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Nushell support#6330
base:master
Are you sure you want to change the base?
Nushell support#6330
Conversation
Thanks for the PR!
If you don't want to add full support, just need a json output to be able to populate nushell environment, you can take a look tothis comment, we have in mind to have something similar for CIs.
The other places where a change is needed are for generating the init scripts. they are generated at
If nushell users are willing to tell us when there is some update to do, and what are they, this is ok. It is hard for us to maintain a shell we don't use :). |
@@ -76,6 +76,7 @@ users) | |||
## Clean | |||
## Env | |||
* [NEW] Add support for nushell in the opam env command |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
*[NEW] Add support for nushell in the opam env command | |
*[NEW] Add support for nushell in the opam env command[#6330@benjamin-voisin] |
This change make the command
opam env
work withinnushell by typingIt does not add full nushell support, only for the
env
command, but it’s the first step. I don’t know that much about the code of opam and could not figure out where to make the other changes neccessary / when they are called, so this is open to improvments.NOTE: nushell still has some breaking changes sometimes, so I don’t know how this will hold up to the future, but I am certain that it will be easier to change an existing implementation.