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

Added ignore_terminal setting to allow bypassing the message "NVM For… #1071

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

aaron777collins
Copy link

@aaron777collins aaron777collins commented Dec 7, 2023

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).

… 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).
Copy link

github-actions bot commented Jan 7, 2024

This PR is stale because it has been open 45 days with no activity.

@github-actions github-actions bot added the Stale Stale label Jan 7, 2024
@coreybutler coreybutler removed the Stale Stale label Jan 7, 2024
aaron777collins and others added 4 commits January 11, 2024 15:05
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).
@aaron777collins
Copy link
Author

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.

Copy link

This PR is stale because it has been open 45 days with no activity.

@github-actions github-actions bot added the Stale Stale label Feb 22, 2024
Copy link

@VersoriumX VersoriumX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge

@github-actions github-actions bot removed the Stale Stale label Apr 8, 2024
@aaron777collins
Copy link
Author

I've removed the artifacts from my forked repo (README changes) so that this can be merged :)

@aaron777collins
Copy link
Author

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.

@aaron777collins
Copy link
Author

Should close #1068 if merged. Allows users to opt-in to checking the terminal (by setting the new property to false).

Copy link

@VersoriumX VersoriumX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed

Copy link

@VersoriumX VersoriumX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

git commit

@coreybutler
Copy link
Owner

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.

Copy link

This PR is stale because it has been open 45 days with no activity.

@github-actions github-actions bot added the Stale Stale label May 16, 2024
@coreybutler coreybutler removed the Stale Stale label May 16, 2024
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.

[Issue]: nvm-update.exe: NVM for Windows should be run from a terminal such as CMD or PowerShell
3 participants