Closed kmohrman closed 8 months ago
Attention: 1 lines
in your changes are missing coverage. Please review.
Comparison is base (
80bfc08
) 28.12% compared to head (5803830
) 27.17%. Report is 1 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
analysis/topeft_run2/analysis_processor.py | 66.66% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Ok, thanks Brent. I've rerun the CI after merging the topcoffea PR as you suggested, and it indeed passes. So I'll go ahead and merge now.
In this PR:
topcoffea
because they are general and not analysis dependent (see topcoffea PR here).coffea<2023
requirement in the environment yml file (because the CI was failing since this repo is still in coffea 0.7).