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

upgrade meck to 0.8.8 #70

Merged
merged 1 commit into from
Aug 30, 2017
Merged

Conversation

connorjacobsen
Copy link
Contributor

@connorjacobsen connorjacobsen commented Aug 29, 2017

eproxus/meck#179 was fixed in the most recent meck release, this should unbreak mock for users with elixir 1.5 / OTP 20

meck PR here: eproxus/meck#180

@connorjacobsen
Copy link
Contributor Author

I can also tweak the mix.lock changes if needed

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling dee8311 on connorjacobsen:upgrade-meck-dep into 8725926 on jjh42:master.

@Olshansk
Copy link
Collaborator

Thanks for the update.

I think you're running an older version of hex given the lock metadata changes. Can you update it via mix local.hex and re-update the mix.lock file?

@connorjacobsen
Copy link
Contributor Author

@Olshansk updated

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 7e2c6dc on connorjacobsen:upgrade-meck-dep into 8725926 on jjh42:master.

@Olshansk Olshansk merged commit f2a3b6e into jjh42:master Aug 30, 2017
@connorjacobsen connorjacobsen deleted the upgrade-meck-dep branch August 30, 2017 17:01
@connorjacobsen
Copy link
Contributor Author

@Olshansk Anything I can do to help get a patch release out with the meck upgrade?

@Olshansk
Copy link
Collaborator

@connorjacobsen Published the release with version 0.3.1.

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

3 participants