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

dep: require nokogiri >= 1.12.0 #267

Merged
merged 1 commit into from
May 11, 2023
Merged

Conversation

flavorjones
Copy link
Owner

Closes #266

This PR updates the Nokogiri dependency to >= 1.12.0, and the Ruby dependency to >= 2.5.0 (which matches the requirement of Nokogiri 1.12.x).

which in turn requires ruby >= 2.5.0, so let's be explicit about that
requirement in the gemspec as well.
@baburdick
Copy link

FYI: If you don't yank v2.21.1 and v2.21.0, bundler will still try to upgrade to them from v2.20.x, for those stuck on nokogiri < v1.12.0.

@flavorjones
Copy link
Owner Author

flavorjones commented May 15, 2023

I'm not yanking these versions of the gem, as yanking tends to be more disruptive to the community. Please pin this dependency in your apps if you're on Ruby 2.4 or earlier, or consider upgrading. Apologies for the inconvenience.

@baburdick
Copy link

Fair enough. But we're on Ruby 2.7. Our OS is unable to upgrade nokogiri. We'll pin on our end. Thanks for the quick reply.

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.

Getting errors using Nokogiri < 1.12
2 participants