diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index a267c124a38e3..ecb0d73c89291 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -1 +1,31 @@ - + + +#### What it does + + +#### How to test + + +#### Review Checklist + + + +- [ ] commits are signed-off: https://github.com/theia-ide/theia/blob/master/CONTRIBUTING.md#sign-your-work +- [ ] commit history is cleaned up of merge and fix up commits + - use `git pull -r` to pick up changes from the master +- [ ] tests/benchmarks are defined for functionallity/perf optimizations and verified by a reviewer, not an author +- [ ] [changelog](https://github.com/theia-ide/theia/blob/master/CHANGELOG.md) is updated +- [ ] breaking changes are justified and recorded in the changelog +- [ ] new dependendencies are justified and [verified](https://github.com/theia-ide/theia/wiki/Registering-CQs#wip---new-ecd-theia-intellectual-property-clearance-approach-experimental) +- [ ] copied code are justified and [approved via a CQ](https://github.com/theia-ide/theia/wiki/Registering-CQs#case-3rd-party-project-code-copiedforked-from-another-project-into-eclipse-theia-maintained-by-us) +- [ ] new code is aligned with [project organization](https://github.com/theia-ide/theia/wiki/Code-Organization) and [coding conventions](https://github.com/theia-ide/theia/wiki/Coding-Guidelines) +- [ ] each review approve is provided with a comment regarding whether functionallity is tested, design or code are aligned with project organization and conventions + - an approve without comment should be dismissed + - a reviewer should focus on requirements and design validation first, testing second and checking code alignment as the last