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 Jul 15, 2022

API methods that return a collection should never return null.

Fixes https://github.com/jenkinsci/code-coverage-api-plugin/issues/426

@uhafner uhafner added the bug Bugs or performance problems label Jul 15, 2022
@codecov
Copy link

codecov bot commented Jul 15, 2022

Codecov Report

Merging #428 (2166351) into master (58ee93a) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master     #428   +/-   ##
=========================================
  Coverage     72.18%   72.18%           
  Complexity      992      992           
=========================================
  Files            86       86           
  Lines          3700     3700           
  Branches        433      433           
=========================================
  Hits           2671     2671           
  Misses          883      883           
  Partials        146      146           
Impacted Files Coverage Δ
...nkins/plugins/coverage/targets/CoverageResult.java 33.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 58ee93a...2166351. Read the comment docs.

@uhafner uhafner merged commit a265014 into master Jul 15, 2022
@uhafner uhafner deleted the npe-additional-properties branch July 15, 2022 08:00
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.

NullPointerException in publishCoverage step after upgrading from 2.0.4 to 3.0.2

2 participants