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.

Sign up for GitHub

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

Jump to bottom

use ~c"" for charlists #757

Merged

bitwalker merged 1 commit into bitwalker:main from simonmcconnell:main

Sep 25, 2024

Contributor

simonmcconnell commented Jun 13, 2024 •

edited

Loading

Summary of changes

I'll review the commits, so I mostly want to understand the "why" rather than the "what"

Checklist

New functions have typespecs, changed functions were updated
Same for documentation, including moduledocs
Tests were added or updated to cover changes
Commits were squashed into a single coherent commit
Notes added to CHANGELOG file which describe changes at a high-level


          use ~c"" for charlists

4aa826d

odarriba commented Sep 11, 2024

Any chances of having this one in to avoid warnings in Elixir 1.17?

zhihuizhang17 commented Sep 14, 2024

I hope this PR gets merged soon.

jadercorrea commented Sep 25, 2024

+1 for this one getting merged.

bitwalker approved these changes

View reviewed changes

bitwalker merged commit 01ad66b into bitwalker:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet