Skip to content

Add synced pull request template#25

Merged
nicolejms merged 1 commit into
mainfrom
brooke-hamilton-add-pr-template-sync
Jul 10, 2026
Merged

Add synced pull request template#25
nicolejms merged 1 commit into
mainfrom
brooke-hamilton-add-pr-template-sync

Conversation

@brooke-hamilton

Copy link
Copy Markdown
Member

Summary

Adds a standard pull_request_template.md to sync-templates/.github/ and wires it into .github/sync.yml so it is distributed to all radius-project repositories via the existing repo-files-sync mechanism.

Reason for change

The organization has no shared PR template, so PR descriptions are inconsistent across repos. Adding a synced template gives every repo the same standard structure and context for reviewers.

How to test

  1. Confirm sync-templates/.github/pull_request_template.md exists and renders as expected.
  2. Confirm the new mapping in .github/sync.yml points sync-templates/.github/pull_request_template.md.github/pull_request_template.md in the all-repos group.
  3. After merge to main, the sync workflow opens PRs in target repos adding .github/pull_request_template.md.

File change summary

File Summary of change
sync-templates/.github/pull_request_template.md New PR template with Summary, Reason for change (optional issue link), How to test, and File change summary sections.
.github/sync.yml Added the template to the all-repos sync group so it propagates to every target repository.

Add a standard pull_request_template.md to sync-templates/.github and wire it into .github/sync.yml so it propagates to all radius-project repositories. The template includes Summary, Reason for change (with optional issue link), How to test, and File change summary sections.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@brooke-hamilton brooke-hamilton added the pr:standard Standard pull request label Jul 10, 2026
@brooke-hamilton
brooke-hamilton marked this pull request as draft July 10, 2026 22:33
@brooke-hamilton
brooke-hamilton marked this pull request as ready for review July 10, 2026 22:35
@brooke-hamilton
brooke-hamilton requested a review from Copilot July 10, 2026 22:36
@nicolejms
nicolejms merged commit 129b412 into main Jul 10, 2026
4 checks passed
@nicolejms
nicolejms deleted the brooke-hamilton-add-pr-template-sync branch July 10, 2026 22:37

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

Adds a standardized pull request template to the sync-templates set and updates the repo-files-sync configuration so the template is propagated to all radius-project repositories, improving consistency of PR descriptions across the org.

Changes:

  • Added sync-templates/.github/pull_request_template.md with standard PR sections (Summary, Reason for change, How to test, File change summary).
  • Updated .github/sync.yml to sync the new PR template into target repositories at .github/pull_request_template.md.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
sync-templates/.github/pull_request_template.md Introduces the org-wide PR template content to be distributed.
.github/sync.yml Adds a sync mapping so the PR template is propagated to the all-repos group.

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

Comment on lines +14 to +18
Explain why this change is needed. If it addresses a GitHub issue, link it
below so it is automatically closed when this PR merges (optional).
-->

Fixes #<!-- issue number (optional) -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:standard Standard pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants