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

feat: add TestResultsFlow for measuring time to notification #439

Merged
merged 2 commits into from
Jun 19, 2024

fix: remove redundant test results subflow

adc9560
Select commit
Loading
Failed to load commit list.
Merged

feat: add TestResultsFlow for measuring time to notification #439

fix: remove redundant test results subflow
adc9560
Select commit
Loading
Failed to load commit list.
Codecov Public QA / codecov/project succeeded Jun 19, 2024 in 0s

97.46% (+0.00%) compared to a2522d2

View this Pull Request on Codecov

97.46% (+0.00%) compared to a2522d2

Details

Codecov Report

Attention: Patch coverage is 95.83333% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 97.46%. Comparing base (a2522d2) to head (adc9560).

✅ All tests successful. No failed tests found ☺️

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #439   +/-   ##
=======================================
  Coverage   97.46%   97.46%           
=======================================
  Files         417      417           
  Lines       34796    34819   +23     
=======================================
+ Hits        33913    33936   +23     
  Misses        883      883           
Components Coverage Δ
NonTestCode 94.48% <95.83%> (+<0.01%) ⬆️
OutsideTasks 97.74% <100.00%> (+<0.01%) ⬆️
Files Coverage Δ
helpers/checkpoint_logger/flows.py 100.00% <100.00%> (ø)
tasks/test_results_finisher.py 95.23% <100.00%> (+0.23%) ⬆️
tasks/tests/unit/test_upload_task.py 99.49% <ø> (ø)
tasks/upload.py 91.06% <85.71%> (-0.14%) ⬇️

... and 1 file with indirect coverage changes