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

add support for Ruby 3.4 in tests #3257

Merged
merged 2 commits into from
Jan 9, 2025
Merged

add support for Ruby 3.4 in tests #3257

merged 2 commits into from
Jan 9, 2025

Conversation

joelhawksley
Copy link
Contributor

What are you trying to accomplish?

In order to update ViewComponent's CI to run Ruby 3.4 when testing against Primer ViewComponents, we need to add support for Ruby 3.4 in tests here. This is probably a good idea regardless due to GitHub.com using Ruby 3.4.

Risk Assessment

  • Low risk the change is small, highly observable, and easily rolled back.

Merge checklist

  • Added/updated tests
  • Added/updated documentation
  • Added/updated previews (Lookbook)
  • Tested in Chrome
  • Tested in Firefox
  • Tested in Safari
  • Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

@joelhawksley joelhawksley requested a review from a team as a code owner January 9, 2025 21:27
@joelhawksley joelhawksley requested a review from jonrohan January 9, 2025 21:27
Copy link

changeset-bot bot commented Jan 9, 2025

🦋 Changeset detected

Latest commit: 1939fa8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/view-components Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@jonrohan jonrohan merged commit 9f87ab1 into main Jan 9, 2025
34 checks passed
@jonrohan jonrohan deleted the ruby3.4 branch January 9, 2025 22:14
@primer primer bot mentioned this pull request Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants