feat(workflows): add submodules input to reusable CI workflows#18
Conversation
Adds an optional `submodules` input to `required.yml`, `dotnet-ci.yml`, and `security-scan.yml`. Forwards to `actions/checkout` in the build, format, test, and CodeQL jobs. Empty default preserves current behavior for all existing callers. Unblocks repos that depend on git submodules (e.g. resq-software/viz's `lib/dotnet-sdk`). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
|
Warning Rate limit exceeded
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 4 minutes and 55 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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 configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✨ 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 |
Summary
submodulesinput torequired.yml,dotnet-ci.yml, andsecurity-scan.ymlactions/checkoutin build, format, test, and CodeQL jobs"") preserves current behavior for all existing callersWhy
resq-software/vizdepends onlib/dotnet-sdkas a git submodule, but CI's reusable workflows never check it out — causing build and CodeQL failures withCS0246for every SDK-sourced type. This change lets consumer repos opt into submodule checkout via a single input.Caller usage (viz example)
Backwards compatibility
Purely additive. The
submodulesinput defaults to"", which when forwarded toactions/checkoutis treated identically to "not set" — no existing caller (resQ,vcpkg,viz,dotnet-sdk) changes behavior unless they explicitly opt in.Test plan
resq-software/viz/.github/workflows/ci.ymland passsubmodules: recursivegates / .NET CI / Buildandscan / CodeQL (csharp)pass on viz PR with a valid submodule pin🤖 Generated with Claude Code