Closed mirekdlugosz closed 1 year ago
Merging #446 (ad6ff57) into main (9641fe0) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## main #446 +/- ##
=======================================
Coverage 79.11% 79.11%
=======================================
Files 5 5
Lines 249 249
=======================================
Hits 197 197
Misses 52 52
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
As DISCOVERY-426 showed, sometimes removing scan that is still in progress results in 500 error from server. This is rare (about 5% of tries) and a bit of edge case (why would you try to delete a scan that is in progress?), but may lowers our confidence in PR pipelines.
Since this never happens during standard (finished) scan removal, the easiest way forward is to wait for scans to finish. And if we have to wait for scans to finish, we might as well check in UI if we can download them.