-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
When using nushell with --no-history I get error on oh-my-posh.nu #4887
Comments
Unfortunately, this is not really related to oh-my-posh... So I've been able to duplicate this problem within a fresh Ubuntu installation (via Docker) and have Nushell produce the same output without oh-my-posh installed: Your question seems a little oxymoronic. @JanDeDobbeleer, in my opinion, this can be closed. |
@ivan-the-terrible we can always check if history is available. If not, default to something sensible and not crash the initialization. |
That makes sense. I didn't realize oh-my-posh officially supported nushell. What would be the course of action? To raise an issue or attempt to put out a PR there? |
@ivan-the-terrible I don't mind solving this one. Nushell is my daily driver anyways 😃 |
Sounds good, I'm very curious to see the result! |
Hey @JanDeDobbeleer, you maight try checking |
Man I am so stupid. I didn't realize this was actually affecting code here in the codebase 🤦🏼 I thought it was just an issue with the way the |
@ivan-the-terrible hope you learned something! |
This issue has been automatically locked since there has not been any recent activity (i.e. last half year) after it was closed. It helps our maintainers focus on the active issues. |
Code of Conduct
What happened?
When trying to use a nushell with history deactivated with nu --no-history
I get the following:
history is null and this statement fails.
Theme
the error does not depend on the theme
What OS are you seeing the problem on?
Windows
Which shell are you using?
nu
Log output
The text was updated successfully, but these errors were encountered: