Skip to content

feat: implement issue #350 — Compliance: sonar-s7637-exemption-missing#360

Open
don-petry wants to merge 1 commit into
mainfrom
dev-lead/issue-350-20260702-2057
Open

feat: implement issue #350 — Compliance: sonar-s7637-exemption-missing#360
don-petry wants to merge 1 commit into
mainfrom
dev-lead/issue-350-20260702-2057

Conversation

@don-petry

Copy link
Copy Markdown
Contributor

Closes #350

Implemented by dev-lead agent. Please review.

@don-petry don-petry requested a review from a team as a code owner July 2, 2026 21:02
Copilot AI review requested due to automatic review settings July 2, 2026 21:02
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@don-petry, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 37 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 9694b750-95fa-4203-8c06-7318e428b961

📥 Commits

Reviewing files that changed from the base of the PR and between 71c3ba0 and 84ee7ad.

📒 Files selected for processing (9)
  • .github/workflows/add-to-project.yml
  • .github/workflows/agent-shield.yml
  • .github/workflows/auto-rebase.yml
  • .github/workflows/dependabot-automerge.yml
  • .github/workflows/dependabot-rebase.yml
  • .github/workflows/dependency-audit.yml
  • .github/workflows/dev-lead.yml
  • .github/workflows/pr-auto-review.yml
  • .github/workflows/pr-review-mention.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev-lead/issue-350-20260702-2057

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

Comment @coderabbitai help to get the list of available commands.

@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@sonarqubecloud

sonarqubecloud Bot commented Jul 2, 2026

Copy link
Copy Markdown

@don-petry

Copy link
Copy Markdown
Contributor Author

Dev-Lead — review-changes (no-changes)

No changes were needed for this PR.

@don-petry don-petry enabled auto-merge (squash) July 2, 2026 21:03
@donpetry-bot

Copy link
Copy Markdown
Contributor

Advisory bots were rate-limited; auto-approval is withheld until they recover. pr-review-sweep will re-review this PR after 2026-07-02T22:04:22Z.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates several GitHub Actions thin-caller workflow stubs and related inline documentation in an attempt to address issue #350 (SonarCloud githubactions:S7637 exemption for first-party reusable workflow refs).

Changes:

  • Normalizes/adjusts YAML formatting in multiple workflow stubs (notably around secrets: / secrets: inherit blocks).
  • Adds/updates inline documentation in dev-lead.yml about SonarCloud S7637 handling.
  • Fixes the referenced “Standard” anchor in dependency-audit.yml.

Reviewed changes

Copilot reviewed 2 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.github/workflows/pr-review-mention.yml YAML formatting adjustment around secrets: inherit.
.github/workflows/pr-auto-review.yml YAML formatting adjustment in secrets: mapping.
.github/workflows/dev-lead.yml Adds SonarCloud S7637 explanatory comments; adjusts standard reference text.
.github/workflows/dependency-audit.yml Updates the “Standard” comment anchor; minor formatting alignment.
.github/workflows/dependabot-rebase.yml YAML formatting adjustment in secrets: mapping.
.github/workflows/dependabot-automerge.yml YAML formatting adjustment around secrets: inherit.
.github/workflows/auto-rebase.yml YAML formatting adjustment around secrets: inherit.
.github/workflows/agent-shield.yml Minor formatting alignment for the uses: line.
.github/workflows/add-to-project.yml YAML formatting adjustment in secrets: mapping.

Comment on lines +11 to +16
# 5. SonarCloud-gated repo? Nothing to do — the `uses:` line below already
# carries the inline `# NOSONAR(githubactions:S7637)` marker. The ref is a
# first-party reusable pinned to a moving channel/ring tag, so without the
# marker SonarCloud's githubactions:S7637 would fail the Quality Gate. The
# marker travels with this file on copy; no sonar-project.properties entry
# is needed. See ../ci-standards.md#sonarcloud-exemption-first-party-reusable-ref-s7637.
# dependency). Promotion is done by moving the dev-lead/stable tag centrally; this
# caller is never edited on release. agent_ref threads the same channel into
# dev-lead's own scripts/prompts checkout. See ci-standards.md#dev-lead-agent.
# dev-lead's own scripts/prompts checkout. See ../ci-standards.md#dev-lead-agent.
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.

Compliance: sonar-s7637-exemption-missing

3 participants