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

Recommendation 4.2.2 type alias #288

Merged
merged 9 commits into from
Sep 17, 2020

Conversation

kentr0w
Copy link
Collaborator

@kentr0w kentr0w commented Sep 15, 2020

What's done:

Recommendation 4.2.2 type alias
Closes #293

Actions checklist

  • Implemented Rule, added Warnings
  • Added tests on checks
  • Updated diktat-analysis.yml
  • Updated available-rules.md

### What's done:
    Added new rule
@kentr0w kentr0w added the enhancement New feature or request label Sep 15, 2020
@codecov
Copy link

codecov bot commented Sep 15, 2020

Codecov Report

Merging #288 into master will increase coverage by 0.00%.
The diff coverage is 83.33%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #288   +/-   ##
=========================================
  Coverage     81.30%   81.31%           
- Complexity      960      967    +7     
=========================================
  Files            50       51    +1     
  Lines          2519     2531   +12     
  Branches        789      793    +4     
=========================================
+ Hits           2048     2058   +10     
  Misses          187      187           
- Partials        284      286    +2     
Flag Coverage Δ Complexity Δ
#unittests 81.31% <83.33%> (+<0.01%) 967.00 <7.00> (+7.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> (ø)
...lin/org/cqfn/diktat/ruleset/rules/TypeAliasRule.kt 80.00% <80.00%> (ø) 7.00 <7.00> (?)
...tlin/org/cqfn/diktat/ruleset/constants/Warnings.kt 96.42% <100.00%> (+0.04%) 8.00 <0.00> (ø)
...cqfn/diktat/ruleset/rules/DiktatRuleSetProvider.kt 97.56% <100.00%> (+0.06%) 3.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 8bec6f1...5e42ec8. Read the comment docs.

### What's done:
    Added soe tests
    Changed warn's message
### What's done:
    Added handler for generic count
    Added tests
### What's done:
    Added tags for tests
### What's done:
    Fixed after merged
### What's done:
    Added comments
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, but take a look at comments

### What's done:
  Added KDoc
  Added to another ymls file
@kentr0w kentr0w merged commit 0e5efcf into master Sep 17, 2020
@kentr0w kentr0w deleted the feature/recommendation-4.2.2-type-alias branch September 17, 2020 15:00
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.

Recommendation 4.2.2: Try to use type alias to represent types and make code more readable
2 participants