Skip to content
Open
Changes from 1 commit
Commits
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
31 changes: 16 additions & 15 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,22 +7,23 @@
[Describe the steps needed to test this]

## Checklist
- [ ] My code follows the style guide.
- [ ] I have self-reviewed my code.
- [ ] I added comments for hard-to-understand code.
- [ ] If applicable, PHP code is documented using PHPDoc.
- [ ] If applicable, JavaScript code is documented using JSDoc.
- [ ] If needed, the ELMO Guide has been updated.
- [ ] If needed, the README has been updated.
- [ ] If needed, the API documentation has been updated.
- [ ] If a new feature was added or a bug fixed, the changelog has been updated.
#### My code is styled, reviewed and understandable
- [ ] PHP code is documented using PHPDoc.
- [ ] JavaScript code is documented using JSDoc.

#### My code is documented
- [ ] ELMO Guide (`./doc/help.html`) has been updated.
- [ ] README has been updated.
- [ ] API documentation (`./api/v2/docs`) has been updated.
- [ ] changelog (`./doc/changelog.html`) has been updated.

#### My code is productive
This PR is covered with the tests. Tests are added first for code that runs often, has a high failure impact, or is difficult to validate through manual testing.
- [ ] PHPUnit
- [ ] JS
- [ ] Playwright
- [ ] My changes do not create new warnings in the test browser console.
- [ ] I have added unit tests that cover my code.
- [ ] All new and existing unit tests pass locally.
- [ ] All new and existing automated unit tests pass in the pull request.
- [ ] If applicable, Playwright tests have been updated and new tests added.
- [ ] All new and existing automated Playwright tests pass in the pull request.
- [ ] I ensured the changes meet accessibility guidelines.
- [ ] My code meets accessibility guidelines.


## Known Issues
Expand Down
Loading