-
Notifications
You must be signed in to change notification settings - Fork 8
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
feat(cat-voices): Enable the 'arithmetic_side_effects' Clippy lint #1456
Conversation
✅ Test Report | |
I think this change is good, but we need to update catalyst-ci to make the flag standard in the standardized rust configs, and use that version. |
I think we should proceed with this change. |
✅ Test Report | |
✅ Test Report | |
I will update the pull request once again when input-output-hk/catalyst-ci#365 is merged. |
42b98fa
to
6078b14
Compare
✅ Test Report | |
✅ Test Report | |
Description
arithmetic_side_effects
Clippy lint was enabled and the code was updated accordingly.Description of Changes
In our style guide it is mentioned that the
integer_arithmetic
lint must be enabled, but this lint was renamed toarithmetic_side_effects
. I decided to see how disruptive can it be, but surprisingly the diff is rather small. I can updatecatalyst-ci
andcatalyst-libs
repositories as well as the corresponding part of the style guide if we decided to proceed with enabling this lint. I think it is useful because it actually highlights a potential panic in theCatalystRBACTokenV1::is_young
function.Please confirm the following checks