-
-
Notifications
You must be signed in to change notification settings - Fork 58
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
glob-watcher is no longer used by gulp #24
Comments
See lodash/lodash#1686 for additional information |
Duplicate of #22 - please search for issues before opening another one. |
@SgtPooki I think you're making this a bigger deal than it needs to be. Facts:
The issue you are referencing has nothing to do with gulp or glob-watcher or any of this: the lodash deprecation warnings are because the author of lodash made a poor decision. The issue was closed because @phated believes that other people's dumb choices are not ours to fix. It looks like the deprecation warnings have been removed (from the issue you linked) and your problem has been solved so I'm not sure what you're looking for here. |
@phated Please stop locking issues - every time you lock an issue people start emailing me instead. |
@contra correction, the deprecation warning was not removed but it actually means nothing to end users. The issue was opened because there was a person complaining it caused windows to exit with error code > 0. However, after a discussion with npm and microsoft, it was determined that user was incorrect and the error code was still 0 when npm writes to stderr on windows. A deprecation warning was still an uninformed and mistaken move by lodash but it doesn't seem like it will be removed. The warning doesn't mean anything though because there aren't bugs in the code our deps of deps of deps of deps are using. The issues get locked because there is nothing else we can do and I am sick of being attacked in threads by someone blowing shit out of proportion. Sorry that users don't understand what that means and email you. |
@phated Understand, better to unwatch issues than lock though or people get frustrated and write angry tweets/emails about how we don't care about what they have to say and are total dickheads. |
So, this package should be deprecated or removed since it's contributing to a dependency chain that contains other deprecated packages.
I opened another issue (#23) for this, but it was closed and muted without any attempt to actually resolve or inform the public about the issue to help reduce these contacts from continuing to come in. I reached out to @contra privately and after informing me that
he said I should
I completely agree, and my response to his email was as follows:
The text was updated successfully, but these errors were encountered: