-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Added ignore_terminal setting to allow bypassing the message "NVM For… #1071
base: master
Are you sure you want to change the base?
Added ignore_terminal setting to allow bypassing the message "NVM For… #1071
Conversation
… Windows should be run from a terminal such as CMD or Powershell." when you are using git bash (for people who want 1.1.11 behaviour).
This PR is stale because it has been open 45 days with no activity. |
Pulling in latest changes
… Windows should be run from a terminal such as CMD or Powershell." when you are using git bash (for people who want 1.1.11 behaviour).
…llins/nvm-windows into Allow_Ignore_Terminal
Please note that the most recent changes in this PR were meant for my own fork. They unintentionally made it in this PR afterwards. However, the original commit should work well as a fix. |
This PR is stale because it has been open 45 days with no activity. |
I've removed the artifacts from my forked repo (README changes) so that this can be merged :) |
Here's a link to a release I made a while ago if y'all want to see how the changes function. This is before I updated the fork to include the new 1.1.13 changes. Treat it as if it were 1.1.12. |
Should close #1068 if merged. Allows users to opt-in to checking the terminal (by setting the new property to false). |
Hey @VersoriumX - stop posting pointless comments. We don't use trigger keywords to merge anything. The plan is to revert the changes back to the original state. There was only one user who wanted this and it was for an easily avoidable situation that doesn't impact most people (i.e. clicking on the icon doe nothing... that was the point of terminal recognition). A new release is held up by the fact our SSL cert got locked. I'm working with the vendor to unlock it. |
This PR is stale because it has been open 45 days with no activity. |
This PR is stale because it has been open 45 days with no activity. |
This PR is stale because it has been open 45 days with no activity. |
Closes #1068
Solves the issue "… Windows should be run from a terminal such as CMD or Powershell." when you are using git bash (for people who want 1.1.11 behaviour).