Skip to content

Add retries to CI - #966

Merged
jpdillingham merged 1 commit into
masterfrom
ci-retries
Aug 30, 2026
Merged

Add retries to CI#966
jpdillingham merged 1 commit into
masterfrom
ci-retries

Conversation

@jpdillingham

Copy link
Copy Markdown
Owner

There are a few flaky unit tests so I'm adding retries around the test step. This PR gets rid of the ci script and moves the steps within it to the workflow definition so I can avoid re-building and re-initializing sonarscanner every iteration and instead just re-run tests.

@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Aug 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.16%. Comparing base (fd439ba) to head (3263b22).
⚠️ Report is 16 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #966      +/-   ##
==========================================
- Coverage   99.18%   99.16%   -0.03%     
==========================================
  Files         185      189       +4     
  Lines        6632     6704      +72     
  Branches     1012     1024      +12     
==========================================
+ Hits         6578     6648      +70     
- Misses         41       42       +1     
- Partials       13       14       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jpdillingham
jpdillingham merged commit 28e68eb into master Aug 30, 2026
4 of 5 checks passed
@jpdillingham
jpdillingham deleted the ci-retries branch August 30, 2026 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant