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

Bugfix. CompactInitializationRule builds incorrect tree(#599) #600

Merged
merged 18 commits into from
Dec 8, 2020

Conversation

aktsay6
Copy link
Collaborator

@aktsay6 aktsay6 commented Dec 1, 2020

What's done:

  • Fixed bug

This pull request closes #599

### What's done:
  * Fixed bug
@aktsay6 aktsay6 added the bug Something isn't working label Dec 1, 2020
@aktsay6 aktsay6 requested a review from petertrr December 1, 2020 16:00
@codecov
Copy link

codecov bot commented Dec 1, 2020

Codecov Report

Merging #600 (af8da65) into master (c73048a) will increase coverage by 0.00%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #600   +/-   ##
=========================================
  Coverage     80.75%   80.75%           
- Complexity     1745     1748    +3     
=========================================
  Files            89       89           
  Lines          4613     4615    +2     
  Branches       1385     1386    +1     
=========================================
+ Hits           3725     3727    +2     
  Misses          322      322           
  Partials        566      566           
Flag Coverage Δ Complexity Δ
unittests 80.75% <75.00%> (+<0.01%) 0.00 <0.00> (ø)

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

Impacted Files Coverage Δ Complexity Δ
...tat/ruleset/rules/classes/CompactInitialization.kt 81.81% <75.00%> (-1.00%) 17.00 <0.00> (+2.00) ⬇️
.../cqfn/diktat/ruleset/rules/files/WhiteSpaceRule.kt 79.59% <0.00%> (+0.68%) 129.00% <0.00%> (+1.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 c73048a...af8da65. Read the comment docs.

# Conflicts:
#	diktat-rules/src/main/kotlin/org/cqfn/diktat/ruleset/rules/classes/CompactInitialization.kt
### What's done:
  * Fixed bugs
### What's done:
  * Added test to smoke tests
### What's done:
  * Added test to smoke tests
### What's done:
  * Fixed bugs
### What's done:
  * Fixed bugs
@aktsay6 aktsay6 requested review from petertrr and orchestr7 December 8, 2020 09:14
aktsay6 and others added 11 commits December 8, 2020 12:35
### What's done:
  * Fixed bugs
### What's done:
  * Fixed bugs
### What's done:
  * Fixed bugs
### What's done:
  * Fixed bugs
### What's done:
  * Fixed bugs
### What's done:
  * Fixed bugs
### What's done:
  * Fixed bugs
### What's done:
  * Fixed bugs
### What's done:
  * Fixed bugs
@aktsay6 aktsay6 merged commit ac07d98 into master Dec 8, 2020
@aktsay6 aktsay6 deleted the bugfix/compact-incorrect-tree(#599) branch December 8, 2020 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CompactInitializationRule builds incorrect tree
3 participants