feat(actions): add retention days for uploaded artifacts in build and…

[samtrion]

… test workflows

Summary by CodeRabbit

• Chores
  • Optimized CI/CD artifact retention policies for improved efficiency—pull request artifacts now retain for 1 day while others retain for 7 days.
  • Streamlined test pipeline artifact management by consolidating uploads and adjusting retention settings for coverage and log files.

[coderabbitai]

Warning

Rate limit exceeded

@samtrion has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 26 minutes and 10 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 26 minutes and 10 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 47e2f91a-bd2d-47b2-a9ac-0d3ed4e6c047

📥 Commits

Reviewing files that changed from the base of the PR and between f3cf854 and 61a1eb3.

📒 Files selected for processing (2)

• .github/workflows/step-dotnet-build.yml
• .github/workflows/step-dotnet-tests.yml

Walkthrough

Two GitHub Actions workflow steps were changed: the build workflow sets artifact retention conditionally by event (pull_request = 1 day, others = 7 days); the tests workflow removed a ReportGenerator upload, narrowed coverage upload globs with retention-days: 3, and changed log upload to run only on failures with retention-days: 3.

Changes

Cohort / File(s)	Summary
Build workflow artifact retention .github/workflows/step-dotnet-build.yml	NuGet package upload step now sets retention-days conditionally: 1 day for pull_request events, 7 days for other event types. No other inputs or artifact naming changed.
Test workflow artifact management .github/workflows/step-dotnet-tests.yml	Removed the "Upload ReportGenerator" artifact step. "Upload Coverage Reports" now uploads only **/TestResults/**/*.opencover.xml and **/TestResults/**/*.cobertura.xml with retention-days: 3. "Upload log files" now runs if: failure() (was always()) and sets retention-days: 3; log/SARIF globs unchanged.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I hopped through workflows, tidy and spry,
Changing days where artifacts lie.
Pull requests nap after a single sun,
Coverage and logs keep watch when tests are undone.
A little rabbit clap—clean builds, well done! 🎀

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}