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 String.slice deprecation warning in Elixir >= 1.16 #97

Conversation

urmastalimaa
Copy link
Contributor

String.slice(1..-1) was hard-deprecated in Elixir 1.16. Fix the warning by giving String.slice(term, 1..-1//1).

`String.slice(1..-1)` was [hard-deprecated in Elixir 1.16][changelog].
Fix the warning by giving `String.slice(term, 1..-1//1)`.

[changelog]: https://github.com/elixir-lang/elixir/releases/tag/v1.16.0
@urmastalimaa urmastalimaa requested a review from a team as a code owner June 13, 2024 12:06
@urmastalimaa
Copy link
Contributor Author

I added bump to version 2.1.1 if that suits you. bc1edc0

@davydog187
Copy link
Member

There are other pending changes. Please don't include version bumps in PRs, let maintainers handle

@urmastalimaa urmastalimaa force-pushed the fix_string_slice_deprecation_warning branch from bc1edc0 to d4f2c10 Compare June 13, 2024 19:50
@urmastalimaa
Copy link
Contributor Author

Sure thing, removed

@davydog187 davydog187 merged commit f4091e2 into beam-community:master Jul 2, 2024
2 checks passed
@halfdan
Copy link

halfdan commented Jul 31, 2024

@davydog187 Any timeline on a new release?

@davydog187
Copy link
Member

@halfdan i tried releasing a new version but there's some org secres issues https://github.com/beam-community/avro_ex/actions/runs/10182889703

Working with @doomspork to get this resolved

@davydog187
Copy link
Member

v2.2.0 has been released

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