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

Add metric parameter for CyclomaticComplexity #6

Merged
merged 2 commits into from
Mar 31, 2023
Merged

Conversation

uhafner
Copy link
Member

@uhafner uhafner commented Mar 31, 2023

Otherwise serialized instances are wrongly mapped.

@uhafner uhafner added the bug Bugs or performance problems label Mar 31, 2023
@codecov
Copy link

codecov bot commented Mar 31, 2023

Codecov Report

Merging #6 (a05fd29) into main (899761a) will decrease coverage by 0.04%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##               main       #6      +/-   ##
============================================
- Coverage     85.23%   85.20%   -0.04%     
- Complexity      478      479       +1     
============================================
  Files            23       23              
  Lines          1321     1325       +4     
  Branches        193      193              
============================================
+ Hits           1126     1129       +3     
- Misses          150      151       +1     
  Partials         45       45              
Impacted Files Coverage Δ
src/main/java/edu/hm/hafner/coverage/Value.java 28.12% <0.00%> (-0.91%) ⬇️
...a/edu/hm/hafner/coverage/CyclomaticComplexity.java 100.00% <100.00%> (ø)
src/main/java/edu/hm/hafner/coverage/Metric.java 89.33% <100.00%> (+0.14%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@uhafner uhafner merged commit bf0bfe6 into main Mar 31, 2023
@uhafner uhafner deleted the max-serialization branch March 31, 2023 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugs or performance problems
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant