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

Update req dependency to allow 0.5.x #16

Merged
merged 3 commits into from
Jun 13, 2024

Conversation

fastjames
Copy link
Contributor

@fastjames fastjames commented Jun 10, 2024

In order to maintain compatibility with recent releases of other gems (e.g. swoosh), update the req dependency to allow 0.5.x.

In order to maintain compatibility with recent releases of other gems
(e.g. swoosh), update the req dependency to 0.5.x.
mix.exs Outdated Show resolved Hide resolved
@benwilson512
Copy link
Member

Also, this doesn't seem to be passing. Can you please make whatever additional changes are required to get it to work on 0.5.x ?

@fastjames
Copy link
Contributor Author

Also, this doesn't seem to be passing. Can you please make whatever additional changes are required to get it to work on 0.5.x ?

Yes, I'll take a look at that now.

@fastjames
Copy link
Contributor Author

Also, this doesn't seem to be passing. Can you please make whatever additional changes are required to get it to work on 0.5.x ?

It looks like the failure occurs when testing with elixir 1.12, because newer versions req only support 1.13 and above. Specifically, the use of `Keyword.validate!/2 is what causes the error (because it's not available in elixir 1.12).

I will update the CI matrix to account for the ineligible version.

Change CI and mix.exs to indicate that elixir versions 1.13 and above
are now supported (due to the req version update).
@benwilson512 benwilson512 merged commit fe9df99 into CargoSense:main Jun 13, 2024
2 of 3 checks passed
@fastjames fastjames deleted the fastjames/chore/req_0_5 branch June 13, 2024 20:29
@fastjames
Copy link
Contributor Author

Thanks for the merge!

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