-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[arithmetic-side-effects
]: Consider user-provided pairs
#9840
Conversation
@Alexendoo #9592 has been merged. |
We talked about this in the meeting the other day, one thing that came up was that we could use |
|
7ac5c39
to
8aa2023
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.
Just a couple code style things and we can get this merged (once bors becomes unstuck)
Thanks! @bors r+ |
☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test |
Depends on #9592.
Because of #9559 (comment), r? @Alexendoo