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

Kdoc fixer false-negative: not generating a Kdoc for several public functions #989

Merged
merged 7 commits into from
Jul 19, 2021

Conversation

Cheshiriks
Copy link
Member

@Cheshiriks Cheshiriks commented Jul 16, 2021

What's done:

@Cheshiriks Cheshiriks requested a review from petertrr July 16, 2021 15:31
@Cheshiriks Cheshiriks force-pushed the bugfix/Kdoc-fixer-false-negative branch from 347b8ba to 9ae14fd Compare July 16, 2021 15:33
@codecov
Copy link

codecov bot commented Jul 16, 2021

Codecov Report

Merging #989 (469d1ea) into master (4ac0f89) will increase coverage by 0.03%.
The diff coverage is 88.88%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #989      +/-   ##
============================================
+ Coverage     83.81%   83.85%   +0.03%     
- Complexity     2409     2416       +7     
============================================
  Files           102      102              
  Lines          6081     6094      +13     
  Branches       1799     1802       +3     
============================================
+ Hits           5097     5110      +13     
  Misses          269      269              
  Partials        715      715              
Flag Coverage Δ
unittests 83.85% <88.88%> (+0.03%) ⬆️

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

Impacted Files Coverage Δ
...leset/rules/chapter2/comments/HeaderCommentRule.kt 86.59% <ø> (ø)
...at/ruleset/rules/chapter3/files/IndentationRule.kt 85.91% <ø> (ø)
...tlin/org/cqfn/diktat/ruleset/utils/KotlinParser.kt 89.28% <ø> (ø)
.../diktat/ruleset/rules/chapter2/kdoc/KdocMethods.kt 89.91% <88.88%> (+1.23%) ⬆️

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 4ac0f89...469d1ea. Read the comment docs.

@Cheshiriks Cheshiriks force-pushed the bugfix/Kdoc-fixer-false-negative branch from bd52ae0 to bbc57ec Compare July 16, 2021 15:45
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.

Please provide a more detailed description in the PR, to summarize what we are changing here. We are changing criterion for getters and setters and changing expression body detection, right?
Also please check out and fix newly discovered rules violations in diktat_snapshot workflow

…unctions

### What's done:
* fixed bug in MISSING_KDOC_ON_FUNCTION
Closes #965
…unctions

### What's done:
* fixed bug in MISSING_KDOC_ON_FUNCTION
Closes #965
…unctions

### What's done:
* fixed bug in MISSING_KDOC_ON_FUNCTION
Closes #965
…unctions

### What's done:
* fixed bug in MISSING_KDOC_ON_FUNCTION
Closes #965
…unctions

### What's done:
* fixed bug in MISSING_KDOC_ON_FUNCTION
Closes #965
…unctions

### What's done:
* fixed bug in MISSING_KDOC_ON_FUNCTION
Closes #965
…unctions

### What's done:
* fixed bug in MISSING_KDOC_ON_FUNCTION
Closes #965
@Cheshiriks Cheshiriks force-pushed the bugfix/Kdoc-fixer-false-negative branch from 1520b81 to 469d1ea Compare July 19, 2021 16:04
@Cheshiriks Cheshiriks merged commit 9e3fde2 into master Jul 19, 2021
@kgevorkyan kgevorkyan deleted the bugfix/Kdoc-fixer-false-negative branch July 20, 2021 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Kdoc fixer false-negative: not generating a Kdoc for several public functions
2 participants