-
Notifications
You must be signed in to change notification settings - Fork 445
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
Revert infinite loop bug #286
base: master
Are you sure you want to change the base?
Conversation
46209a8
to
1b47a63
Compare
1b47a63
to
02e6e4a
Compare
@@ -1,7 +1,7 @@ | |||
{ | |||
"name": "colors", | |||
"description": "get colors in your node.js console", | |||
"version": "1.4.44-liberty-2", | |||
"version": "1.5.0", |
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.
better use 1.4.5 in order to updated all
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.
The problematic previous version is 1.4.44 (-liberty-2). So it would have to be 1.4.45 if you think it needs to be a minor version. Was just attempting to make it simple for people to avoid all of the 1.4.X code given the severity of this.
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.
Check colors/safe
as well, 1.4.2 also has the same issue.
LGTM ✔️ |
I'm unable to merge or npm publish this. Is a maintainer able to do that? |
It won't be merged. The author did this on purpose, it's not a mistake nor a bug. They did similar things on their other packages, such as Faker.js (they removed the source, and push v6.6.6 to NPM that breaks everything). |
Don't merge this, it breaks functionality of the new library and how I use it. |
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.
Stop breaking people's code!!! If you want, log a polite message, but dOS is not acceptable!
This reverts a critical error described here: 074a0f8