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

fix: catch ruby3.3 format NoMethodError message #1579

Merged
merged 1 commit into from
Dec 1, 2023

Conversation

mtasaka
Copy link
Contributor

@mtasaka mtasaka commented Nov 5, 2023

ruby3.3.0dev changes the error messages on NoMethodError with the following commit / issue:

ruby/ruby#6950
https://bugs.ruby-lang.org/issues/18285

So to catch this new NoMethodError message, modify regex accordingly.

Fixes #1578 .

ruby3.3.0dev changes the error messages on NoMethodError with the following
commit / issue:

ruby/ruby#6950
https://bugs.ruby-lang.org/issues/18285

So to catch this new NoMethodError message, modify regex accordingly.

Fixes thoughtbot#1578 .
Copy link
Member

@matsales28 matsales28 left a comment

Choose a reason for hiding this comment

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

LGTM! We currently do not support Ruby 3.3, but this shouldn't affect the application, so I'm okay with it.

Thanks for your contribution!

@matsales28 matsales28 merged commit 3654e5c into thoughtbot:main Dec 1, 2023
9 checks passed
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.

rspec spec/unit fails with ruby 3.3.0dev
2 participants