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

Bump dependencies #31

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Bump dependencies #31

wants to merge 2 commits into from

Conversation

axelson
Copy link
Contributor

@axelson axelson commented Mar 1, 2019

earmark 0.2.1 -> 1.3.1
ex_doc 0.11.5 -> 0.19.3
idna 5.1.0 -> 6.0.0

Judging by the idna changelog, there aren't any actual breaking changes
in 6.0.0:
https://github.com/benoitc/erlang-idna/blob/3eb54ccbfa6fb917c0f4ca9197da337ad888ffe0/CHANGELOG

Turns out that idna 6.0.0 has some breaking changes so adjust how
punycode conversions are done

References:

Note: the test "compiles using a newly fetched copy of the rules file if so configured" seems to use a version of publicsuffix-elixir from the deps directory so I couldn't get it to pass locally.

earmark 0.2.1 -> 1.3.1
ex_doc 0.11.5 -> 0.19.3
idna 5.1.0 -> 6.0.0

Judging by the idna changelog, there aren't any actual breaking changes
in 6.0.0:
https://github.com/benoitc/erlang-idna/blob/3eb54ccbfa6fb917c0f4ca9197da337ad888ffe0/CHANGELOG

Turns out that idna 6.0.0 has some breaking changes so adjust how
punycode conversions are done

References:
* benoitc/erlang-idna#28
* benoitc/erlang-idna#26 (comment)
@axelson
Copy link
Contributor Author

axelson commented Mar 1, 2019

Should probably test this new code with previous idna versions, but not sure if there's a way to do that in exunit. Maybe by specifying an extra mix.lock file for testing?

@ghost
Copy link

ghost commented Jul 25, 2019

@axelson @seomoz anything I can do to help get this merged? the current version of idna conflicts with hackney.

@axelson
Copy link
Contributor Author

axelson commented Apr 6, 2020

@r3l4t1v you can just use my fork: https://github.com/axelson/publicsuffix-elixir

Seomoz hasn't been responsive for quite a while. I may push my fork to hex at some point.

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.

1 participant