Skip to content

Remove Ruby 2.7 from test matrix; update Nokogiri v1.16.2 - #146

Merged
bensheldon merged 2 commits into
mainfrom
update-nokogiri
Feb 6, 2024
Merged

Remove Ruby 2.7 from test matrix; update Nokogiri v1.16.2#146
bensheldon merged 2 commits into
mainfrom
update-nokogiri

Conversation

@bensheldon

Copy link
Copy Markdown
Contributor

Ruby 2.7 is EOL and thus it can't be expected that dependencies will continue to support it.

Nokogiri (which is a development dependency) has a security bulletin and requires updating.

@bensheldon
bensheldon requested a review from a team as a code ownerFebruary 6, 2024 16:30
Comment thread.github/workflows/ci.yml Outdated
fail-fast: false
matrix:
ruby_version: ["2.7", "3.0", "3.1", "3.2"]
ruby_version: ["3.0", "3.1", "3.2"]

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.

Can we add 3.3 while we're here as well?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

👍🏻

@mattr-mattr- 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.

:shipit:

@bensheldon
bensheldon merged commit cbfc24a into mainFeb 6, 2024
@bensheldon
bensheldon deleted the update-nokogiri branch February 6, 2024 17:39
@abrahamabraham mentioned this pull request Feb 12, 2024
Sign up for freeto 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

@bensheldon@mattr-