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

Updated version of glob package to latest version #2123

Closed
wants to merge 2 commits into from

Conversation

dmsakamoto
Copy link

NPM says glob dependency graceful-fs@2.0.3 is deprecated and will cause newer versions of Node to crash.
screen shot 2016-02-24 at 1 14 04 pm
So I upgraded glob from version @3.2.3 to @7.0.0, which is the latest version. This version of glob also no longer uses graceful-fs as a dependency.

Finally, this update passed all of the tests.
screen shot 2016-02-24 at 1 28 42 pm
mochaTestOutput.txt

@RobertWHurst
Copy link

Can someone kindly merge this PR please.

@SamuelMarks
Copy link

Actually can you bump that 3 patch versions? - https://github.com/isaacs/node-glob/blob/2fc2278ab857c7df117213a2fb431de090be6353/package.json

7.0.3 is out

@aldeed
Copy link

aldeed commented Mar 8, 2016

+1 This outdated dep also causes issues with the upcoming new release of Meteor, requiring a bunch of workarounds to test Meteor apps. (ref meteor/meteor#6430 (comment))

cc @boneskull @danielstjules

@dmsakamoto
Copy link
Author

@SamuelMarks I updated glob to 7.0.3

@dmsakamoto dmsakamoto closed this Mar 8, 2016
@dmsakamoto
Copy link
Author

There was a duplicate (#2144) that was merged

@aldeed
Copy link

aldeed commented Mar 8, 2016

@dmsakamoto #2144 was not merged. It was closed as dup of this issue.

@RobertWHurst
Copy link

@danielstjules
Copy link
Contributor

Hey all, can we move the discussion to #2121 ? :)

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.

None yet

5 participants