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

[DEPENDENCY] Pin dir-glob to v2.0.0 #73

Merged
merged 1 commit into from
Jan 10, 2019
Merged

[DEPENDENCY] Pin dir-glob to v2.0.0 #73

merged 1 commit into from
Jan 10, 2019

Conversation

RandomByte
Copy link
Member

dir-glob is a transitive dependency of globby ("dir-glob": "^2.0.0").

Some of our tests ("GLOB root directory" and "GLOB (normalized) root
directory (=> fs root)") are failing when using dir-glob v2.2.0.

Therefore pinning it to v2.0.0 for the time being.

This came up in our preflight tests which ignore the package-lock.json:
npm install --no-shrinkwrap --no-package-lock

Note: This is unrelated to #71
Tests were still failing after reverting that change.

Moreover, this is also reproducible on Git tag v0.2.0.

dir-glob is a transitive dependency of globby ("dir-glob": "^2.0.0").

Some of our tests ("GLOB root directory" and "GLOB (normalized) root
directory (=> fs root)") are failing when using dir-glob v2.2.0.

Therefore pinning it to v2.0.0 for the time being.

This came up in our preflight tests which ignore the package-lock.json:
npm install --no-shrinkwrap --no-package-lock

Note: This is unrelated to #71
Tests were still failing after reverting that change.

Moreover, this is also reproducible on Git tag v0.2.0.
@RandomByte RandomByte requested a review from matz3 January 10, 2019 00:46
@RandomByte RandomByte merged commit b921fbc into master Jan 10, 2019
@RandomByte RandomByte deleted the pin-dir-glob-dep branch January 10, 2019 06:50
RandomByte added a commit that referenced this pull request Jan 30, 2019
Follow up of #73
To pin a transitive dependency when using Yarn, "resolutions" need to be
used.
RandomByte added a commit that referenced this pull request Jan 31, 2019
Follow up of #73
To pin a transitive dependency when using Yarn, "resolutions" need to be
used.
@RandomByte RandomByte mentioned this pull request Nov 19, 2020
6 tasks
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