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
Prevent invalid cause from breaking exception presenter #2703
Prevent invalid cause from breaking exception presenter #2703
Conversation
Don't merge this yet as I think something is wrong |
4898582
to
7bc3332
Compare
7bc3332
to
dda873b
Compare
@pirj this is ready for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. LGTM!
Minor and optional notes for the spec.
dda873b
to
0d5fde4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Green! |
acf6bb0
to
4e81d92
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Good catch @pirj, I'd forgotten to check on this 😂 I redid your fix slightly and rebased. |
…s-invalid-cause Prevent invalid cause from breaking exception presenter
…gnores-invalid-cause Prevent invalid cause from breaking exception presenter
…eption-presenter-ignores-invalid-cause Prevent invalid cause from breaking exception presenter --- This commit was imported from rspec/rspec-core@52c3747.
Fixes #2701