From 76f87396e93a89ad6667123428f6e02456f8e19f Mon Sep 17 00:00:00 2001 From: Boddepalli naveen <1234naveenboddepalli@gmail.com> Date: Sun, 7 Jun 2026 20:01:05 +0530 Subject: [PATCH] updated pr_template.md --- .github/pull_request_template.md | 78 ++++++++++++++++++++------------ 1 file changed, 48 insertions(+), 30 deletions(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index f5a79033b..c2fa23a3b 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -1,65 +1,83 @@ ## Summary -Enhanced the existing `PULL_REQUEST_TEMPLATE.md` to improve contribution quality, accessibility awareness, and reviewer guidance. + -Closes #1107 +Closes # --- ## Type of Change -- [ ] Bug fix -- [ ] New feature -- [x] Documentation update -- [ ] Refactor / code cleanup + + +- [ ] ๐Ÿ› Bug fix (non-breaking change that fixes an issue) +- [ ] โœจ New feature (non-breaking change that adds functionality) +- [ ] ๐Ÿ’ฅ Breaking change (fix or feature that changes existing behavior) +- [ ] ๐Ÿ“ Documentation update +- [ ] โ™ป๏ธ Refactor / code cleanup (no functional change) +- [ ] โšก Performance improvement +- [ ] ๐Ÿ”’ Security fix +- [ ] ๐Ÿงช Tests only --- -## Changes Made +## What Changed + + -- Improved PR template structure and readability -- Added accessibility checklist section -- Added additional notes section -- Enhanced contributor guidance for testing and review -- Improved consistency for future pull requests +- +- +- --- ## How to Test -Steps for the reviewer to verify this works: + + +1. +2. +3. -1. Create a new pull request -2. Verify the updated PR template appears automatically -3. Check that all checklist sections render properly -4. Ensure markdown formatting works correctly +**Expected result:** --- -## Screenshots (if UI change) +## Screenshots / Recordings -N/A + + +| Before | After | +|--------|-------| +| | | --- ## Checklist -- [x] Linked issue in summary -- [x] `npm run lint` passes locally -- [x] No TypeScript errors (`npm run type-check`) -- [x] Self-reviewed the diff -- [ ] Added/updated tests if applicable + + +- [ ] Linked the related issue above +- [ ] Self-reviewed my own diff +- [ ] No unnecessary `console.log`, debug code, or commented-out blocks +- [ ] `npm run lint` passes locally +- [ ] No TypeScript errors (`npm run type-check`) +- [ ] Added or updated tests where applicable +- [ ] Updated documentation / comments if behavior changed --- -## Accessibility Checklist +## Accessibility (UI changes only) + + -- [x] Proper keyboard navigation tested -- [x] Responsive UI verified -- [x] Accessibility labels added where needed +- [ ] Keyboard navigation works correctly +- [ ] Color contrast meets WCAG AA standard +- [ ] ARIA labels / roles added where needed +- [ ] Tested on mobile / responsive layout --- -## Additional Notes +## Additional Context -This update standardizes pull request submissions and helps maintain consistent review quality across contributions. + \ No newline at end of file