Skip to content

Comments

Update to Nokogiri v1.12#211

Merged
rgrove merged 1 commit intorgrove:mainfrom
KrisKotlarek:remove-nokogumbo
Aug 4, 2021
Merged

Update to Nokogiri v1.12#211
rgrove merged 1 commit intorgrove:mainfrom
KrisKotlarek:remove-nokogumbo

Conversation

@KrisKotlarek
Copy link
Contributor

Nokogiri v1.12 merged Nokogumbo code https://github.com/sparklemotion/nokogiri/releases/tag/v1.12.0

Therefore, Nokogumbo is not required as a dependency anymore.

If tests are run with latest Nokogiri, note about deprecation is printed:

NOTE: nokogumbo: Using Nokogiri::HTML5 provided by Nokogiri. See https://github.com/sparklemotion/nokogiri/issues/2205 for more information.

Nokogiri v1.12 merged Nokogumbo code https://github.com/sparklemotion/nokogiri/releases/tag/v1.12.0

Therefore, Nokogumbo is not required as a dependency anymore.

If tests are run with latest Nokogiri deprecation note is printed:
```
NOTE: nokogumbo: Using Nokogiri::HTML5 provided by Nokogiri. See sparklemotion/nokogiri#2205 for more information.
```
Copy link
Owner

@rgrove rgrove left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I'll merge this soon.

@rgrove rgrove merged commit c638f38 into rgrove:main Aug 4, 2021
@v29neil
Copy link

v29neil commented Nov 25, 2022

hello, whats the supported nokogiri(nokogumbo merged) version for Ruby 2.3.0 ?
i couldnt find any, can someone help me ?

@rgrove
Copy link
Owner

rgrove commented Nov 25, 2022

Hey @v29neil. Nokogiri 1.12.0 requires Ruby 2.5.0 or higher.

I'm going to lock this conversation because old PRs aren't the best place to ask questions, but if you have more questions feel free to start a discussion.

Repository owner locked as resolved and limited conversation to collaborators Nov 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants