Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Nushell #78

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add Nushell #78

wants to merge 1 commit into from

Conversation

giggio
Copy link

@giggio giggio commented Sep 28, 2023

Adding Nushell to the script was thanks to the work done earlier to setup Fish.
I also updated the readme.

I left out the GNU watch comment, as Nushell does not allow for dynamic eval. If we want to support that we would need to generate 2 files, one with the default watch and the other with the GNU tool.

@ahmetb
Copy link
Owner

ahmetb commented Sep 29, 2023

I'm inclined to say maybe fish shell was a good place to stop :)

@giggio
Copy link
Author

giggio commented Sep 30, 2023

I have the PowerShell one just waiting for this PR. :)
Have you seen how many forks this repo has, adding other shells? It seems to be a common need. It would be nice to have other shells being supported. It would be cool to have this centralized in this repo.
But I understand if you want to keep it more focused, I'll use my fork, then. Feel free to close if that is the direction you want to move.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants