Skip to content

Loosen public suffix restriction to allow version 5.0#460

Merged
dentarg merged 2 commits intosporkmonger:mainfrom
petergoldstein:feature/upgrade_public_suffix_version_to_allow_5
Jul 25, 2022
Merged

Loosen public suffix restriction to allow version 5.0#460
dentarg merged 2 commits intosporkmonger:mainfrom
petergoldstein:feature/upgrade_public_suffix_version_to_allow_5

Conversation

@petergoldstein
Copy link
Copy Markdown
Contributor

public_suffix 5.0 was just released. It includes updated definitions, and is restricted to Ruby 2.6+.

Loosening this gemspec restriction allows addressable to use the latest version of public_suffix when loaded with a supported Ruby (2.6+).

public_suffix 5.0 was just released.  It includes updated definitions, and is restricted to Ruby 2.6+.

Loosening this restriction allows addressable to use the latest version of public_suffix when loaded with a supported Ruby.
@petergoldstein
Copy link
Copy Markdown
Contributor Author

CI failures appears to be unrelated, because the same Windows/JRuby 9.3 failures are happening in this PR - #459 - as well.

@dentarg
Copy link
Copy Markdown
Collaborator

dentarg commented Jul 25, 2022

Change looks fine to me, no breaking API changes in public_suffix what I can see:

@dentarg dentarg merged commit d2df5e8 into sporkmonger:main Jul 25, 2022
@dentarg
Copy link
Copy Markdown
Collaborator

dentarg commented Jul 25, 2022

Totally forgot the fact that there's a rake task to generate the gemspec 🤦

I'll update it

dentarg added a commit that referenced this pull request Jul 25, 2022
dentarg added a commit to dentarg/addressable that referenced this pull request Jul 25, 2022
I ran "bundle exec rake gem:gemspec" and this is what changed, except
the date (did not change the date as I think that should reflect when
releases are made).

Related to sporkmonger#460 and sporkmonger#463 that both updated the generated gemspec
directly. Includes the change from sporkmonger#425.
@dentarg dentarg mentioned this pull request Jul 25, 2022
dentarg added a commit that referenced this pull request Jul 25, 2022
I ran "bundle exec rake gem:gemspec" and this is what changed, except
the date (did not change the date as I think that should reflect when
releases are made).

Related to #460 and #463 that both updated the generated gemspec
directly. Includes the change from #425.
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