ci: add push workflow and update README badges with package table#286
Conversation
Reviewer's GuideAdds a new GitHub Actions workflow that builds the solution on pushes to main and updates the README to use the new workflow’s status badge and to present NuGet packages in a table with individual version badges. Flow diagram for new GitHub Actions push workflowflowchart TD
A[Push to main branch] --> B[GitHub Actions Push workflow]
B --> C[Checkout repository actions/checkout@v6]
C --> D[Setup .NET SDK actions/setup-dotnet@v5]
D --> E[dotnet restore --locked-mode]
E --> F[dotnet build --configuration Release --no-restore]
File-Level Changes
Assessment against linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've found 1 issue, and left some high level feedback:
- If you want the new
push.ymlstatus badge to reflect the health of incoming changes as well, consider adding apull_requesttrigger alongsidepushso the workflow also runs for PRs targetingmain.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- If you want the new `push.yml` status badge to reflect the health of incoming changes as well, consider adding a `pull_request` trigger alongside `push` so the workflow also runs for PRs targeting `main`.
## Individual Comments
### Comment 1
<location path=".github/workflows/push.yml" line_range="35-36" />
<code_context>
+ - name: Restore .NET Packages
+ run: dotnet restore --locked-mode
+
+ - name: Build .NET Solution
+ run: dotnet build --configuration Release --no-restore
</code_context>
<issue_to_address>
**suggestion (testing):** Consider adding a test step to catch regressions on push to main
Since this runs on `main`, consider adding a `dotnet test --configuration Release --no-build` step after the build so CI verifies the test suite as well as compilation.
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (3 files)
Reviewed by step-3.7-flash-20260528 · 189,358 tokens |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
WalkthroughAdded a GitHub Actions push workflow for the main branch that builds the solution in Release mode with .NET SDK caching and locked dependency restoration. Updated the README to reference the new workflow in build badges and added a NuGet Packages section documenting four published packages. ChangesPush Automation
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Closes #285
Summary by Sourcery
Add a push workflow that builds the .NET solution on pushes to main and update the README to reflect the new workflow and package offerings.
Build:
Documentation:
Summary by CodeRabbit
Documentation
Chores