Skip to content

Expand PR validation template #162

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mehul0810 merged 1 commit into from
Jul 4, 2026
Merged

Expand PR validation template #162

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
82 changes: 62 additions & 20 deletions .github/PULL_REQUEST_TEMPLATE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,20 +1,62 @@
## Description
<!-- Please describe your changes -->

## How Has This Been Tested?
<!-- Please describe in detail how you tested your changes. -->
<!-- Include details of your testing environment, tests ran to see how -->
<!-- your change affects other areas of the code, etc. -->

## Screenshots (jpeg or gifs if applicable):

## Types of changes
<!-- What types of changes does your code introduce? -->
<!-- Bug fix (non-breaking change which fixes an issue) -->
<!-- New feature (non-breaking change which adds functionality) -->
<!-- Breaking change (fix or feature that would cause existing functionality to not work as expected) -->

## Checklist:
- [ ] My code is tested.
- [ ] My code follows the WordPress code style.
- [ ] My code follows has proper inline documentation.
## Linked issue
<!-- Link the issue this PR closes or explain why there is no issue. -->

Closes #

## Target branch
<!-- Confirm the intended base branch, such as develop or release/1.7.0. -->

- [ ] I confirmed this PR targets the correct base branch.

## Scope
<!-- Summarize the change and keep the boundary clear. -->

### Summary

### Non-goals
<!-- List related work intentionally left out, or write "Not applicable". -->

### User-facing impact
<!-- Describe visible behavior/UI/docs changes, or write "Not applicable". -->

## Validation
<!-- Check every gate you ran. Mark irrelevant gates as "Not applicable - reason". -->

- [ ] PHPCS / code style:
- [ ] PHP lint:
- [ ] PHPStan:
- [ ] PHPUnit:
- [ ] Frontend lint/build:
- [ ] Playwright / browser proof:
- [ ] WordPress smoke tests:
- [ ] Manual testing:

## Screenshots or video
<!-- Required for UI/workflow changes. For non-UI changes, explain why proof is not applicable. -->

## Compatibility
<!-- Mark items as "Not applicable - reason" when they are unrelated to this PR. -->

- Settings/options/backward compatibility:
- Public hooks or public PHP API:
- Migrations and existing installs:
- Multisite:
- Supported PHP and WordPress versions:
- Security/privacy-sensitive behavior:
- Documentation/readme/changelog/release notes:

## Contributor PR handling
<!-- Fill this out when this PR replaces, salvages, or follows up on a human contributor PR. Otherwise write "Not applicable". -->

- Contributor PR:
- What was preserved from the contributor work:
- Credit note:
- Why a maintainer replacement is needed:

## Checklist

- [ ] The PR has a clear linked issue, scope, and non-goals.
- [ ] I avoided unrelated formatting, lockfile, or generated-file churn.
- [ ] I documented skipped validation gates with a concrete reason.
- [ ] UI changes include screenshot/video proof or a reason proof is not applicable.
- [ ] Contributor credit is preserved where applicable.
Loading