Skip to content

Commit e085d76

Browse files
committed
Docs: complete cut-off sentence in 'Preparing a pull request' section
1 parent b8f2e8f commit e085d76

File tree

1 file changed

+11
-1
lines changed

1 file changed

+11
-1
lines changed

contributor_docs/preparing_a_pull_request.md

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,17 @@ Then ask git about the latest changes.
5151
git status
5252

5353
## Pull Request Templates
54-
Once you've opened your pull request, please ensure that you follow the guidelines.
54+
55+
Once you've opened your pull request, please ensure that you follow the guidelines **and verify the following before submitting for review:**
56+
57+
- ✅ Your code has no linting errors (`npm run lint`)
58+
- ✅ All tests pass successfully (`npm run test`)
59+
- ✅ Your branch name is unique and based off the latest `develop` branch
60+
- ✅ The pull request title and description clearly describe the change
61+
- ✅ Your pull request links to the related issue number (e.g., `Fixes #1234`)
62+
- ✅ Screenshots or videos are included if the change affects the user interface
63+
64+
Following these steps helps maintain code quality and makes it easier for maintainers to review and merge your contribution.
5565

5666
## CONFLICTS
5767
You may have some conflicts! It's okay. Feel free to ask for help. If merging with the latest upstream `develop` branch causes conflicts, you can always make a pull request with the upstream repository, which makes the merge conflicts public.

0 commit comments

Comments
 (0)