Skip to content

initial nokogiri libgumbo integration - #11004

Merged
jonathanmetzman merged 7 commits into
google:masterfrom
fuzzy-boiii23a:nokogiri_integration
Oct 4, 2023
Merged

initial nokogiri libgumbo integration#11004
jonathanmetzman merged 7 commits into
google:masterfrom
fuzzy-boiii23a:nokogiri_integration

Conversation

@fuzzy-boiii23a

Copy link
Copy Markdown
Contributor

criticality score: 0.75414

approval: sparklemotion/nokogiri#2992

Please let me know if there is anything else that needs changing and just to note i'll be creating a pull request to integrate the harness into the upstream nokogiri repo sometime this week.

@github-actions

Copy link
Copy Markdown

fuzzy-boiii23a is integrating a new project:
- Main repo: https://github.com/sparklemotion/nokogiri.git
- Criticality score: 0.75692

Comment thread projects/nokogiri/project.yaml Outdated
@jonathanmetzman

Copy link
Copy Markdown
Contributor

We approve of adding thsi project. You just need to add a real user first.

Update primary contact
@fuzzy-boiii23a

Copy link
Copy Markdown
Contributor Author

We approve of adding thsi project. You just need to add a real user first.

Good to go now, thanks mate :)

@jonathanmetzman jonathanmetzman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Almost everything looks good

Comment thread projects/nokogiri/gumbo.dict
Comment thread projects/nokogiri/project.yaml
Comment thread projects/nokogiri/project.yaml Outdated
@fuzzy-boiii23a

Copy link
Copy Markdown
Contributor Author

i resolved your comments @jonathanmetzman , please let me know if there are any more improvements i should make, thanks for the help it's much appreciated

@jonathanmetzman jonathanmetzman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@jonathanmetzman
jonathanmetzman enabled auto-merge (squash) October 4, 2023 14:24
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