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

[#1347] Clean up the code #1376

Conversation

0x6675636b796f75676974687562
Copy link
Member

@0x6675636b796f75676974687562 0x6675636b796f75676974687562 commented Jun 20, 2022

What's done:

@codecov
Copy link

codecov bot commented Jun 20, 2022

Codecov Report

Merging #1376 (0a3a32f) into master (b2a52ab) will decrease coverage by 0.06%.
The diff coverage is 77.61%.

@@             Coverage Diff              @@
##             master    #1376      +/-   ##
============================================
- Coverage     83.53%   83.46%   -0.07%     
+ Complexity     2552     2546       -6     
============================================
  Files           106      107       +1     
  Lines          7517     7543      +26     
  Branches       2081     2082       +1     
============================================
+ Hits           6279     6296      +17     
- Misses          359      367       +8     
- Partials        879      880       +1     
Flag Coverage Δ
unittests 83.46% <77.61%> (-0.07%) ⬇️

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

Impacted Files Coverage Δ
.../cqfn/diktat/ruleset/utils/indentation/Checkers.kt 79.48% <ø> (ø)
...otlin/org/cqfn/diktat/ruleset/utils/StringUtils.kt 87.87% <50.00%> (-2.45%) ⬇️
...at/ruleset/rules/chapter3/files/IndentationRule.kt 82.77% <77.77%> (-2.18%) ⬇️
...t/ruleset/rules/chapter3/files/IndentationError.kt 100.00% <100.00%> (ø)
...tat/ruleset/utils/indentation/IndentationConfig.kt 100.00% <100.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 b2a52ab...0a3a32f. Read the comment docs.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 20, 2022

JUnit Tests (macOS, EnricoMi/publish-unit-test-result-action@v1)

1 260 tests   1 240 ✔️  8m 20s ⏱️
   151 suites       20 💤
   151 files           0

Results for commit 0a3a32f.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 20, 2022

JUnit Tests (Windows, EnricoMi/publish-unit-test-result-action@v1)

   151 files     151 suites   9m 13s ⏱️
1 260 tests 1 238 ✔️ 20 💤 2
1 262 runs  1 240 ✔️ 20 💤 2

For more details on these failures, see this check.

Results for commit 0a3a32f.

♻️ This comment has been updated with latest results.

@0x6675636b796f75676974687562 0x6675636b796f75676974687562 force-pushed the feature/andrey.shcheglov/indentation-unit-tests branch 2 times, most recently from d2e9455 to 78b76a4 Compare June 20, 2022 16:14
### What's done:

 * Code cleaned up in preparation to settle #1347.
 * No logic is modified with this change.
### What's done:

 * `IndentationError` moved to a separate source file.
 * Top-level extensions moved from `IndentationRule.kt` to `StringUtils.kt`.
@0x6675636b796f75676974687562 0x6675636b796f75676974687562 force-pushed the feature/andrey.shcheglov/indentation-unit-tests branch from 78b76a4 to 0a3a32f Compare June 20, 2022 16:25
@0x6675636b796f75676974687562 0x6675636b796f75676974687562 deleted the feature/andrey.shcheglov/indentation-unit-tests branch June 21, 2022 06:30
Copy link
Member

@nulls nulls left a comment

Choose a reason for hiding this comment

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

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WRONG_INDENTATION: running mvn diktat:fix adds whitespace to string literals
3 participants