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

Relax dependency constraint on Finch #182

Merged
merged 2 commits into from
Jan 27, 2022
Merged

Conversation

thbar
Copy link
Contributor

@thbar thbar commented Jan 25, 2022

See #181

@coveralls
Copy link

coveralls commented Jan 27, 2022

Coverage Status

Coverage decreased (-1.6%) to 88.764% when pulling ac98d79 on thbar:relax-finch into 88054e8 on parroty:master.

@parroty parroty merged commit fe102c4 into parroty:master Jan 27, 2022
@parroty
Copy link
Owner

parroty commented Jan 27, 2022

Thanks!

@thbar thbar mentioned this pull request Mar 8, 2022
thbar added a commit to etalab/transport-site that referenced this pull request Mar 8, 2022
I removed the need to use the fork by fixing upstream here parroty/exvcr#182
thbar added a commit to etalab/transport-site that referenced this pull request Mar 9, 2022
* Bump all packages

* Go back to exvcr official build

I removed the need to use the fork by fixing upstream here parroty/exvcr#182

* Remove now incorrect comment

See ex-aws/ex_aws#827

* Make sure the message helps me upgrade

* Fix test by fixing deps

* Handle unhandled case

* Revert oban upgrade

* Apply mix format

* Fix bogus test

* Parameterize PLT cache key

This is done in hope to later fix the build by invalidating the cache, which could be responsible for the current failure.

* Try to fix the build
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.

3 participants