-
Notifications
You must be signed in to change notification settings - Fork 1.8k
[missing_panics_doc] Fix #13381
#13382
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
Conversation
|
r? clippy |
|
Looks good, thanks! @bors r+ |
|
☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test |
1 similar comment
|
☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test |
|
👀 Test was successful, but fast-forwarding failed: 422 Changes must be made through a pull request. |
…ly contexts (#15170) changelog: [`missing_panics_doc`]: Allow unwrap() and expect()s in const-only contexts Fixes #15169. As far as I can tell, this change keeps this lint in line with similar issues and their fixes such as #13382 and #10240, so I feel pretty confident that the behavior it exhibits after this PR is correct.
Fix #13381
Makes
missing_panics_docact like other "panicking" lints (unwrap_used,panic, etc) in constant environments.changelog: Ignore
missing_panics_docin constant environments