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

Update NoMethodError message to not use object#inspect #3749

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

rwstauner
Copy link
Collaborator

@rwstauner rwstauner commented Dec 17, 2024

I was using this to run the specs that test against the error message to guide me to something that worked for all the test cases (and let me untag one spec):

jt test `rg -l -g '*_spec.rb' 'undefined.method' spec/`

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Dec 17, 2024
@andrykonchin andrykonchin self-assigned this Dec 18, 2024
@andrykonchin andrykonchin added the in-ci The PR is being tested in CI. Do not push new commits. label Dec 18, 2024
@andrykonchin andrykonchin force-pushed the no-method-error-inspect branch from ca6f8a9 to a6c7e49 Compare December 18, 2024 16:10
@andrykonchin andrykonchin force-pushed the no-method-error-inspect branch from a6c7e49 to 4fe8f1c Compare December 19, 2024 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-ci The PR is being tested in CI. Do not push new commits. OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants