Skip to content

ci: generate skills on release please PRs - #181

Draft
twishabansal wants to merge 5 commits into
mainfrom
ci/skills-gen-forked-renovate-prs
Draft

ci: generate skills on release please PRs#181
twishabansal wants to merge 5 commits into
mainfrom
ci/skills-gen-forked-renovate-prs

Conversation

@twishabansal

@twishabansaltwishabansal commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • generate-skills was gated to same-repo PRs, so it skipped renovate's toolbox bump PRs (which come from a fork) and skills were never regenerated.
  • Switch to pull_request_target and push the regenerated skills back into the renovate PR branch using SKILLS_GEN_TOKEN (a maintainer PAT), since the built-in GITHUB_TOKEN cannot push to a forked PR.
  • Restrict the job to renovate's toolbox bumps (renovate-bot + renovate/googleapis-mcp-toolbox), validate the version is plain semver, and keep the release-please[bot] commit author so cla/google passes.

Run skills generation on pull_request_target for renovate's toolbox bumps
and push the regenerated skills back into the PR branch using a maintainer
PAT, since the built-in GITHUB_TOKEN cannot push to a forked PR.
@twishabansaltwishabansal added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 25, 2026
@twishabansal
twishabansal marked this pull request as draft June 30, 2026 14:01
twishabansaland others added 2 commits June 30, 2026 19:49
Run on release-please's same-repo release PR (release-please--branches--main)
so the built-in GITHUB_TOKEN can push the regenerated skills back into the PR.
Drops pull_request_target, the SKILLS_GEN_TOKEN PAT, and the fork checkout.
@twishabansaltwishabansal changed the title ci: regenerate skills on renovate's forked toolbox PRsci: generate skills on release please PRsJun 30, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not mergeIndicates a pull request not ready for merge, due to either quality or timing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@twishabansal@prernakakkar-google