Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Conversation

@uhafner
Copy link
Member

@uhafner uhafner commented Jun 30, 2022

It makes no sense to keep the broken column for the old model.

@uhafner uhafner added the bug Bugs or performance problems label Jun 30, 2022
@uhafner uhafner linked an issue Jun 30, 2022 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Jun 30, 2022

Codecov Report

Merging #416 (4a27863) into master (68867a2) will increase coverage by 0.48%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #416      +/-   ##
============================================
+ Coverage     71.60%   72.08%   +0.48%     
  Complexity      984      984              
============================================
  Files            87       86       -1     
  Lines          3722     3694      -28     
  Branches        436      431       -5     
============================================
- Hits           2665     2663       -2     
+ Misses          911      885      -26     
  Partials        146      146              
Impacted Files Coverage Δ
...el/visualization/dashboard/CoverageColumnType.java 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 68867a2...4a27863. Read the comment docs.

@uhafner uhafner merged commit b5a5593 into master Jun 30, 2022
@uhafner uhafner deleted the duplicate-coverage-column branch June 30, 2022 21:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Bugs or performance problems

Projects

None yet

Development

Successfully merging this pull request may close these issues.

codecoverage column ambigious with job dsl

2 participants