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

fix missing absolute path resolution in glob version #106

Merged
merged 3 commits into from
Jan 12, 2017

Conversation

zinserjan
Copy link
Owner

mocha-webpack needs glob in version 7.1.0 to get the absolute path option, but the version range for glob in globby is just ^7.0.3 which matches also versions before 7.1.0.

This leads to issues with npm@3's flat dependency tree.

The fastest fix is that we don't rely on the absolute path option.

@zinserjan zinserjan changed the title glob version issue glob version incompatible absolute path with glob versionresolution not available in Jan 12, 2017
@zinserjan zinserjan changed the title glob version incompatible absolute path with glob versionresolution not available in glob version incompatible with absolute path resolution Jan 12, 2017
@zinserjan zinserjan changed the title glob version incompatible with absolute path resolution fix missing absolute path resolution in glob version Jan 12, 2017
@codecov-io
Copy link

Current coverage is 94.60% (diff: 100%)

Merging #106 into master will increase coverage by 0.01%

@@             master       #106   diff @@
==========================================
  Files            26         26          
  Lines           721        723     +2   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            682        684     +2   
  Misses           39         39          
  Partials          0          0          

Powered by Codecov. Last update 593789e...f7a0c9f

@zinserjan zinserjan merged commit a5b2163 into master Jan 12, 2017
@zinserjan zinserjan deleted the glob-version-issue branch January 12, 2017 22:21
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.

2 participants