-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[refurb
] Implement verbose-decimal-constructor
(FURB157
)
#10533
[refurb
] Implement verbose-decimal-constructor
(FURB157
)
#10533
Conversation
|
code | total | + violation | - violation | + fix | - fix |
---|---|---|---|---|---|
FURB157 | 14 | 14 | 0 | 0 | 0 |
Refurb
] Implement simplify-decimal-ctor
(FURB157
)Refurb
] Implement verbose-decimal-constructor
(FURB157
)
Refurb
] Implement verbose-decimal-constructor
(FURB157
)refurb
] Implement verbose-decimal-constructor
(FURB157
)
0f3c5fe
to
8a1b076
Compare
CodSpeed Performance ReportMerging #10533 will degrade performances by 5.31%Comparing Summary
Benchmarks breakdown
|
That looks like a false positive given that the default rule set didn't change here... I'll re-run though. |
8a1b076
to
655821b
Compare
Summary
Implement FURB157 in the issue #1348.
Relevant Refurb docs is here: https://github.com/dosisod/refurb/blob/master/docs/checks.md#furb157-simplify-decimal-ctor
Test Plan
I've written it in the
FURB157.py
.