Skip to content

Modify pull request template for clarity and updates#11776

Merged
zachcasper merged 4 commits into
mainfrom
pr-template-update
May 7, 2026
Merged

Modify pull request template for clarity and updates#11776
zachcasper merged 4 commits into
mainfrom
pr-template-update

Conversation

@zachcasper
Copy link
Copy Markdown
Contributor

Description

Updated PR template after moving design-notes to the Radius repo. Removes samples and recipes from the checklist and replaces with dashboard and resource-types-contrib.

Type of change

  • This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Radius (issue link optional).

Contributor checklist

Please verify that the PR meets the following requirements, where applicable:

  • An overview of proposed schema changes is included in a linked GitHub issue.
    • Yes
    • Not applicable
  • A design document PR is created in the design-notes repository, if new APIs are being introduced.
    • Yes
    • Not applicable
  • The design document has been reviewed and approved by Radius maintainers/approvers.
    • Yes
    • Not applicable
  • A PR for the samples repository is created, if existing samples are affected by the changes in this PR.
    • Yes
    • Not applicable
  • A PR for the documentation repository is created, if the changes in this PR affect the documentation or any user facing updates are made.
    • Yes
    • Not applicable
  • A PR for the recipes repository is created, if existing recipes are affected by the changes in this PR.
    • Yes
    • Not applicable

Updated references to design document PR locations and added checks for resource types and dashboard PRs.

Signed-off-by: Zach Casper <zachcasper@microsoft.com>
Copilot AI review requested due to automatic review settings April 28, 2026 18:42
@zachcasper zachcasper requested review from a team as code owners April 28, 2026 18:42
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 28, 2026

Dependency Review

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

Scanned Files

None

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the repository pull request template to reflect the migration of design notes into this repo and to align cross-repo checklists with the current Radius ecosystem.

Changes:

  • Adds a “design document only” type-of-change option for eng/design-notes/-only PRs.
  • Updates the design document checklist item to point to eng/design-notes/ instead of the old radius-project/design-notes repo.
  • Replaces samples/recipes checklist items with resource-types-contrib and dashboard.

Comment thread .github/pull_request_template.md Outdated
Comment thread .github/pull_request_template.md Outdated
Comment thread .github/pull_request_template.md Outdated
Copy link
Copy Markdown
Member

@brooke-hamilton brooke-hamilton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of modifying the checklist, let's remove the checklist and the require-pr-checklist.yaml workflow.

zachcasper and others added 3 commits May 7, 2026 09:18
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Zach Casper <zachcasper@microsoft.com>
@zachcasper zachcasper merged commit d6cda7c into main May 7, 2026
46 checks passed
@zachcasper zachcasper deleted the pr-template-update branch May 7, 2026 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants