Skip to content
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 6.1.9 #434

Merged
merged 5 commits into from
Oct 23, 2020
Merged

Rule 6.1.9 #434

merged 5 commits into from
Oct 23, 2020

Conversation

kentr0w
Copy link
Collaborator

@kentr0w kentr0w commented Oct 20, 2020

Actions checklist

### What's done:
    Implemented rule
@kentr0w kentr0w added the enhancement New feature or request label Oct 20, 2020
@codecov
Copy link

codecov bot commented Oct 20, 2020

Codecov Report

Merging #434 into master will decrease coverage by 0.03%.
The diff coverage is 76.47%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #434      +/-   ##
============================================
- Coverage     82.67%   82.64%   -0.04%     
- Complexity     1387     1401      +14     
============================================
  Files            67       68       +1     
  Lines          3451     3468      +17     
  Branches       1094     1101       +7     
============================================
+ Hits           2853     2866      +13     
  Misses          175      175              
- Partials        423      427       +4     
Flag Coverage Δ Complexity Δ
#unittests 82.64% <76.47%> (-0.04%) 1401.00 <14.00> (+14.00) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
...at-rules/src/main/kotlin/generated/WarningNames.kt 0.00% <ø> (ø) 0.00 <0.00> (ø)
...qfn/diktat/ruleset/rules/PropertyAccessorFields.kt 73.33% <73.33%> (ø) 14.00 <14.00> (?)
...tlin/org/cqfn/diktat/ruleset/constants/Warnings.kt 97.22% <100.00%> (+0.02%) 9.00 <0.00> (ø)
...cqfn/diktat/ruleset/rules/DiktatRuleSetProvider.kt 96.82% <100.00%> (+0.05%) 8.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3ea3348...cd24be9. Read the comment docs.

### What's done:
    Fixed bugs after review
### What's done:
    Fixed bugs after review
Copy link
Member

@petertrr petertrr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, just one more suggestion

### What's done:
   Renamed variable
…getter-setter-field

# 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
@kentr0w kentr0w merged commit 036f18a into master Oct 23, 2020
@kentr0w kentr0w deleted the feature/rule-6.1.9-getter-setter-field branch October 23, 2020 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rule 6.1.9: never use the name of a variable in the custom getter or setter
2 participants