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.

Sign up for GitHub

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

Jump to bottom

Multiple Deprecations #29

Open
tleen opened this issue Mar 28, 2017 · 3 comments
Open

Multiple Deprecations #29

tleen opened this issue Mar 28, 2017 · 3 comments

Comments

@tleen
Copy link

tleen commented Mar 28, 2017

On install, node 7.0.0:

npm WARN deprecated minimatch@0.2.14: Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated minimatch@0.3.0: Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated graceful-fs@1.2.3: graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to graceful-fs@^4.0.0 as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.
@jgable
Copy link
Owner

jgable commented Mar 28, 2017

Thanks for filing the issue and providing the details on your environment. I don't really use grunt in any of my workflows anymore so I'm not in a good position to maintain this. Would you be willing to put some time into putting a PR together that fixes up those warnings for your environment?

@tleen
Copy link
Author

tleen commented Mar 28, 2017

Sure NP.

tleen pushed a commit to tleen/grunt-phplint that referenced this issue Mar 29, 2017
Update dependencies to current versions including Grunt 1.0.1
bbf493c 
To remove deprecation warnings this updates most of the dependencies. Of course it introduces a known [CoffeeScript related warning](gruntjs/grunt#1556) in Grunt 1.0. This also moves the release version to keep in line with the used Grunt version.

Resolves jgable#29
@tleen tleen mentioned this issue Mar 29, 2017
Closed
@tleen
Copy link
Author

tleen commented Mar 29, 2017

Turns out some of these were from Grunt itself so had to move to the current version. It was a major jump so I bumped version to match. May be a step too much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jgable @tleen