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

Downgrade ktlint #1565

Merged
merged 12 commits into from
Nov 15, 2022
Merged

Downgrade ktlint #1565

merged 12 commits into from
Nov 15, 2022

Conversation

nulls
Copy link
Member

@nulls nulls commented Nov 15, 2022

What's done:

  • downgraded ktlint to 0.46.1

It closes #1564

### What's done:
* downgraded ktlint to 0.46.1

It closes #1564
@nulls nulls self-assigned this Nov 15, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Nov 15, 2022

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

1 975 tests   1 960 ✔️  3m 0s ⏱️
   157 suites       15 💤
   157 files           0

Results for commit 715166d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 15, 2022

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

1 978 tests   1 963 ✔️  2m 7s ⏱️
   161 suites       15 💤
   161 files           0

Results for commit 715166d.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Nov 15, 2022

Codecov Report

Merging #1565 (715166d) into master (341ef78) will decrease coverage by 0.02%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master    #1565      +/-   ##
============================================
- Coverage     82.15%   82.12%   -0.03%     
  Complexity     2314     2314              
============================================
  Files           112      112              
  Lines          7849     7845       -4     
  Branches       2033     2034       +1     
============================================
- Hits           6448     6443       -5     
+ Misses          481      480       -1     
- Partials        920      922       +2     
Flag Coverage Δ
unittests 82.12% <33.33%> (-0.03%) ⬇️

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

Impacted Files Coverage Δ
...kotlin/org/cqfn/diktat/ruleset/rules/DiktatRule.kt 66.66% <ø> (ø)
...in/org/cqfn/diktat/ruleset/rules/OrderedRuleSet.kt 90.32% <ø> (+1.75%) ⬆️
.../cqfn/diktat/ruleset/rules/chapter6/RunInScript.kt 91.17% <ø> (ø)
...lin/org/cqfn/diktat/plugin/maven/DiktatBaseMojo.kt 54.25% <33.33%> (-1.07%) ⬇️
...otlin/org/cqfn/diktat/ruleset/utils/KtlintUtils.kt 85.00% <0.00%> (-5.00%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@nulls nulls enabled auto-merge (squash) November 15, 2022 15:41
@nulls nulls merged commit ff1ec1e into master Nov 15, 2022
@nulls nulls deleted the feature/downgrade-ktlint#1564 branch November 15, 2022 15:47
@nulls nulls added this to the 1.2.4.2 milestone Nov 15, 2022
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.

Downgrade ktlint in master
2 participants