Skip to content

Add general PR review workflow#76

Merged
gontzess merged 1 commit into
mainfrom
gontzess/general-pr-review-workflow
May 13, 2026
Merged

Add general PR review workflow#76
gontzess merged 1 commit into
mainfrom
gontzess/general-pr-review-workflow

Conversation

@gontzess
Copy link
Copy Markdown
Contributor

@gontzess gontzess commented May 13, 2026

Why

Non-connector repositories need the shared PR review ruleset without requiring baton-admin to manage repository Actions variables. The current single entry point defaults to connector review, which is right for connector repos but forces a per-repo override for SDKs and other adjacent repos.

What this changes

Adds a central general-pr-review required workflow that hardcodes the general prompt profile. Refactors the PR review action so prompts are composed from a shared base prompt plus built-in mixins: connector repos still use the existing pr-review workflow and receive the connector mixin by default, while general review uses the base prompt only. Trusted repo-local ci-review skills remain supported for project-specific rules.

@gontzess gontzess force-pushed the gontzess/general-pr-review-workflow branch from 9954a63 to 824e8f2 Compare May 13, 2026 20:47
@gontzess gontzess merged commit 7d301f2 into main May 13, 2026
1 check passed
@gontzess gontzess deleted the gontzess/general-pr-review-workflow branch May 13, 2026 23:54
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.

2 participants