-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Detect t.try() usage in hooks; experimentally disable t.snapshot() #2527
Detect t.try() usage in hooks; experimentally disable t.snapshot() #2527
Conversation
65b9ad1
to
63568bf
Compare
@novemberborn when something is released under the feature flag, when it is a good time to update docs? |
And make it fail the test.
* Rename directory * Snapshot the error message
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.
@okyantoro thanks, I've made a few more tweaks. Let me know what you think.
@tymfear neither of those files mention using snapshots or try() in hooks, so I don't think we need to explicitly point this out. You'll realize when you try to use it.
Thank you @novemberborn |
This change will disable snapshots and try assertions in hooks. But I am not sure with the unit test. Let me know if additional work is required.
Fixes #2523