-
Notifications
You must be signed in to change notification settings - Fork 39
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
Rule 4.1.1 float arithmetic #303
Conversation
### What's done: * Initial implementation * Tests
…float-ariphmethic
### What's done: * Update yml files and available-rules.md
### What's done: * Added new rule to DiktatRuleSetProvider * Added maven central badge
Codecov Report
@@ Coverage Diff @@
## master #303 +/- ##
============================================
- Coverage 82.38% 82.29% -0.10%
- Complexity 1000 1016 +16
============================================
Files 51 52 +1
Lines 2578 2627 +49
Branches 809 830 +21
============================================
+ Hits 2124 2162 +38
Misses 169 169
- Partials 285 296 +11
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
...rules/src/main/kotlin/org/cqfn/diktat/ruleset/rules/calculations/AccurateCalculationsRule.kt
Show resolved
Hide resolved
...rules/src/main/kotlin/org/cqfn/diktat/ruleset/rules/calculations/AccurateCalculationsRule.kt
Outdated
Show resolved
Hide resolved
### What's done: * Fixes
…float-ariphmethic # Conflicts: # diktat-analysis.yml # diktat-rules/src/main/kotlin/generated/WarningNames.kt # diktat-rules/src/main/kotlin/org/cqfn/diktat/ruleset/constants/Warnings.kt # diktat-rules/src/main/resources/diktat-analysis-huawei.yml # diktat-rules/src/main/resources/diktat-analysis.yml # info/available-rules.md
### What's done: * Fixes
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.
What about remainder of dividing (rem, %
)? Or it shouldn't be there?
Fair enough, I forgot about it. |
### What's done: * Added PERC and PERCEQ to handled operators
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.
LGTM
Which rule and warnings did you add?
This pull request closes #289
Actions checklist
* [ ] Added tests on fixersFixme