ci: inline NOSONAR(S7637) marker on dev-lead caller stub (#549 canonical migration)#557
ci: inline NOSONAR(S7637) marker on dev-lead caller stub (#549 canonical migration)#557don-petry wants to merge 44 commits into
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughIn ChangesDev-lead workflow NOSONAR annotation
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
There was a problem hiding this comment.
Pull request overview
Updates the Dev-Lead caller-stub workflow to use the canonical inline SonarCloud suppression marker for rule githubactions:S7637 on the channel-pinned first-party reusable workflow reference, supporting the org-wide migration away from per-repo sonar-project.properties ignore entries while preserving the moving channel tag (@dev-lead/ring0).
Changes:
- Added inline
# NOSONAR(githubactions:S7637)to the first-party reusable workflowuses:line pinned todev-lead/ring0. - Kept the existing permissions configuration and the
@dev-lead/ring0channel pin unchanged.
|
@donpetry-bot please review — S7637 inline-marker migration (#549); markers preserve the channel pins, SonarCloud green. Requesting approval to merge. |
|
@don-petry I'm on it — starting a fresh review now. Results will appear in a few minutes. |
|



Adds the canonical inline
# NOSONAR(githubactions:S7637)marker to the channel-pinned first-party dev-lead caller stub (Option A, #549/#551). Preserves the@dev-lead/ring0channel pin. Part of the controlled fleet migration to inline markers (not the weekly audit). hands-off so no agent re-SHA-pins the channel ref. Legacy per-filesonar-project.propertiesS7637 entries removed in a verified follow-up.Summary by CodeRabbit