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 mingo version so that yarn doesn't upgrade it #34

Closed
wants to merge 1 commit into from

Conversation

balaclark
Copy link

Mingo introduced a breaking change in v1.3.0 (removing the Mingo.setup
method). This causes failures when installing save via mongo when no
yarn.lock is present.

This project uses the new npm 5 package-lock.json, this is not supported
by yarn and so mingo gets silently upgraded from 1.1.2 to 1.3.0. It is
likely that this project will need to upgrade to the mingo 1.3.0 API,
but in the meantime this commit will ensure that 1.1.2 gets installed
when using yarn.

yarnpkg/yarn#3614

Mingo introduced a breaking change in v1.3.0 (removing the Mingo.setup
method). This causes failures when installing save via mongo when no
yarn.lock is present.

This project uses the new npm 5 package-lock.json, this is not supported
by yarn and so mingo gets silently upgraded from 1.1.2 to 1.3.0. It is
likely that this project will need to upgrade to the mingo 1.3.0 API,
but in the meantime this commit will ensure that 1.1.2 gets installed
when using yarn.

yarnpkg/yarn#3614
@serby
Copy link
Owner

serby commented Jul 27, 2017

I've tried to refactor to use the newest. I'm not sure setup was intended to be removed.

kofrasa/mingo#56

@balaclark
Copy link
Author

thanks for opening that issue on mingo, I've re-installed save and the mingo upgrade has sorted the problem, so I'm closing this PR>

@balaclark balaclark closed this Jul 28, 2017
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