Skip to content

ci: dispatch checks for release pull requests - #170

Merged
jerome-benoit merged 3 commits into
masterfrom
ci/dispatch-release-pr-checks
Aug 22, 2026
Merged

ci: dispatch checks for release pull requests#170
jerome-benoit merged 3 commits into
masterfrom
ci/dispatch-release-pr-checks

Conversation

@jerome-benoit

@jerome-benoit jerome-benoit commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

PR Checklist

  • Title follows Conventional Commits.
  • Restores automatic required checks without a custom token.
  • Preserves SonarCloud pull-request analysis on dispatched runs.
  • Uses least-privilege job permissions.
  • Validated workflow structure and formatting.

Does this PR introduce a breaking change?

  • Yes
  • No

Problem

Resources created with the default GITHUB_TOKEN do not emit recursive workflow runs. A plain dispatch can run CI on the release branch, but workflow_dispatch has no native pull-request event context for SonarCloud.

Changes

  • enable workflow_dispatch with required string inputs for PR number, head ref, and base ref
  • grant the Release Please job actions: write plus the documented release permissions
  • extract the three values once from the Release Please PR output
  • dispatch CI with explicit --repo, the release PR head ref, and string-safe --raw-field inputs
  • provide conditional sonar.pullrequest.key, sonar.pullrequest.branch, and sonar.pullrequest.base arguments on dispatched runs

Verification

  • parsed both workflows and asserted the shared dispatch contract, permissions, and conditional Sonar arguments
  • exercised extraction of PR chore(master): release 0.6.4 #144 metadata from the Release Please output shape
  • verified explicit GitHub CLI repository resolution
  • deno task format:check
  • git diff --check

The flow uses the default short-lived token and does not reintroduce a PAT or repository token secret.

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5b868c22d1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/release-please.yml Outdated
@sonarqubecloud

Copy link
Copy Markdown

@jerome-benoit
jerome-benoit merged commit d07567a into master Aug 22, 2026
15 checks passed
@jerome-benoit
jerome-benoit deleted the ci/dispatch-release-pr-checks branch August 22, 2026 15:43
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.

1 participant