diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index c51c8c7c52..5ab971c612 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -2,29 +2,18 @@ -Fixes # (issue) +- Fixes #XXXX (GitHub issue number) +- Fixes CAL-XXXX (Linear issue number - should be visible at the bottom of the GitHub issue description) -## Requirement/Documentation +## Mandatory Tasks (DO NOT REMOVE) - - -- If there is a requirement document, please, share it here. -- If there is a UI/UX design document, please, share it here. - -## Type of change - - - -- Bug fix (non-breaking change which fixes an issue) -- Chore (refactoring code, technical debt, workflow improvements) -- New feature (non-breaking change which adds functionality) -- Breaking change (fix or feature that would cause existing functionality to not work as expected) -- Tests (Unit/Integration/E2E or any other test) -- This change requires a documentation update +- [ ] I have self-reviewed the code (A decent size PR without self-review might be rejected) +- [ ] I have added a Docs issue [here](https://github.com/calcom/docs/issues/new) if this PR makes changes that would require a [documentation change](https://docs.cal.com) +- [ ] I have added or modified automated tests that prove my fix is effective or that my feature works (PRs might be rejected if logical changes are not properly tested) ## How should this be tested? @@ -35,10 +24,6 @@ Fixes # (issue) - What is expected (happy path) to have (input and output)? - Any other important info that could help to test that PR -## Mandatory Tasks - -- [ ] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected. - ## Checklist @@ -46,7 +31,4 @@ Fixes # (issue) - I haven't read the [contributing guide](https://github.com/calcom/cal.com/blob/main/CONTRIBUTING.md) - My code doesn't follow the style guidelines of this project - I haven't commented my code, particularly in hard-to-understand areas -- I haven't checked if my PR needs changes to the documentation - I haven't checked if my changes generate no new warnings -- I haven't added tests that prove my fix is effective or that my feature works -- I haven't checked if new and existing unit tests pass locally with my changes