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

Nokogiri::XML() and Nokogiri::XML.parse() support argument forwarding #3332

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

Conversation

openbl
Copy link

@openbl openbl commented Nov 14, 2024

What problem is this PR intended to solve?

Related to #3323, introducing keyword argument support in Nokogiri::XML::Document.parse.

Have you included adequate test coverage?

Some minor test coverage mimicking the existing permutations of options.

Does this change affect the behavior of either the C or the Java implementations?

No

@flavorjones flavorjones added the event/hackday2024 RubyConf 2024 Hack Day label Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
event/hackday2024 RubyConf 2024 Hack Day
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants