Skip to content

Fix template checker sync#443

Merged
MikeMcQuaid merged 1 commit into
mainfrom
fix-template-checker-sync
Jun 20, 2026
Merged

Fix template checker sync#443
MikeMcQuaid merged 1 commit into
mainfrom
fix-template-checker-sync

Conversation

@MikeMcQuaid

Copy link
Copy Markdown
Member
  • Keep check_template.rb owned by Homebrew/.github instead of syncing it into target repositories.
  • Fetch the checker from Homebrew/.github in synced workflows so targets use shared trusted code.
  • Remove stranded checker copies from synced target repositories.

- Keep `check_template.rb` owned by `Homebrew/.github` instead of
  syncing it into target repositories.
- Fetch the checker from `Homebrew/.github` in synced workflows so
  targets use shared trusted code.
- Remove stranded checker copies from synced target repositories.
Copilot AI review requested due to automatic review settings June 20, 2026 09:52
@MikeMcQuaid MikeMcQuaid enabled auto-merge June 20, 2026 09:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 the “template checker” distribution model so check_template.rb remains centrally owned in Homebrew/.github, while synced workflows in target repositories fetch and run that shared checker, and any previously-synced copies in targets are removed.

Changes:

  • Update check-prs.yml and check-issues.yml to fetch check_template.rb from Homebrew/.github instead of from the target repository.
  • Adjust the sync logic to stop syncing check_template.rb into target repositories and instead remove any existing copies.
  • Ensure the .github repository itself retains its local check_template.rb (not deleted by sync).

Reviewed changes

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

File Description
.github/workflows/check-prs.yml Fetches the template checker script from Homebrew/.github for PR template validation.
.github/workflows/check-issues.yml Fetches the template checker script from Homebrew/.github for issue template validation.
.github/actions/sync/shared-config.rb Stops syncing check_template.rb into targets and removes any copies during sync runs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/check-prs.yml
Comment thread .github/workflows/check-issues.yml
@MikeMcQuaid MikeMcQuaid merged commit 5a45628 into main Jun 20, 2026
28 checks passed
@MikeMcQuaid MikeMcQuaid deleted the fix-template-checker-sync branch June 20, 2026 18:01
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.

3 participants