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

Supported comments after latest enum element #1302

Merged
merged 13 commits into from
May 27, 2022

Conversation

nulls
Copy link
Member

@nulls nulls commented May 25, 2022

What's done:

  • skip EOL_COMMENT, BLOCK_COMMENT, WHITE_SPACE to detect a location for COMMA

This pull request closes #1286

### What's done:
* skip EOL_COMMENT, BLOCK_COMMENT, WHITE_SPACE to detect a location for COMMA
@nulls nulls added this to the 1.1.1 milestone May 25, 2022
@nulls nulls self-assigned this May 25, 2022
@codecov
Copy link

codecov bot commented May 25, 2022

Codecov Report

Merging #1302 (e95c90d) into master (f7c9c8a) will increase coverage by 0.00%.
The diff coverage is 90.90%.

@@            Coverage Diff            @@
##             master    #1302   +/-   ##
=========================================
  Coverage     82.07%   82.08%           
- Complexity     2534     2538    +4     
=========================================
  Files           105      105           
  Lines          7214     7223    +9     
  Branches       2061     2065    +4     
=========================================
+ Hits           5921     5929    +8     
  Misses          346      346           
- Partials        947      948    +1     
Flag Coverage Δ
unittests 82.08% <90.90%> (+<0.01%) ⬆️

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

Impacted Files Coverage Δ
...fn/diktat/ruleset/rules/chapter3/EnumsSeparated.kt 96.42% <90.90%> (-1.45%) ⬇️

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 f7c9c8a...e95c90d. Read the comment docs.

@nulls nulls enabled auto-merge (squash) May 25, 2022 13:04
@nulls nulls requested review from orchestr7 and petertrr May 26, 2022 11:05
nulls added 3 commits May 27, 2022 16:15
### What's done:
* supported KDoc in method for adding new line after each enum element
@nulls nulls disabled auto-merge May 27, 2022 13:17
@nulls nulls enabled auto-merge (squash) May 27, 2022 13:17
@nulls nulls requested a review from petertrr May 27, 2022 13:22
@nulls nulls disabled auto-merge May 27, 2022 13:41
@nulls nulls enabled auto-merge (squash) May 27, 2022 13:41
nulls added 3 commits May 27, 2022 16:47
### What's done:
* amend new line on tree level
### What's done:
* restored logic for newline with whitespace
@nulls nulls merged commit 44507fd into master May 27, 2022
@nulls nulls deleted the bugfix/enum_comma_at_comment#1286 branch May 27, 2022 14:35
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.

ENUMS_SEPARATED fails to converge if last enum entry has comment on same line
2 participants