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

lock `merge`@1.2.1 pkg in package-lock.json #259

Merged
merged 1 commit into from Dec 11, 2018
Merged

Conversation

@cezaraugusto
Copy link
Member

cezaraugusto commented Dec 11, 2018

this is a temp fix while substack/browser-launcher#43 isn't merged

@cezaraugusto cezaraugusto self-assigned this Dec 11, 2018
@bbondy
bbondy approved these changes Dec 11, 2018
@cezaraugusto cezaraugusto merged commit 7c7aa1a into staging Dec 11, 2018
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@cezaraugusto cezaraugusto deleted the merge-pkg-staging branch Dec 11, 2018
cezaraugusto added a commit that referenced this pull request Dec 11, 2018
lock `merge`@1.2.1 pkg in package-lock.json
cezaraugusto added a commit that referenced this pull request Dec 11, 2018
lock `merge`@1.2.1 pkg in package-lock.json
@cezaraugusto
Copy link
Member Author

cezaraugusto commented Dec 11, 2018

master d926cfb
staging 7c7aa1a

@diracdeltas
Copy link
Member

diracdeltas commented Dec 11, 2018

i don't think substack is maintaining this package. @darkdh fixed this issue previously in #245 and we will have to fix this again every time someone updates the deps.

@cezaraugusto unless substack decides to update his package, i think the fix is either to fork browser-launcher, use a different test framework, or add a postinstall script to fix the merge dep every time

@diracdeltas
Copy link
Member

diracdeltas commented Dec 11, 2018

pinged substack and the package has been updated! substack/browser-launcher#42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.