CI: Update PULL_REQUEST_TEMPLATE.md
Signed-off-by: Michael Mayer <michael@photoprism.app>
This commit is contained in:
parent
af14bef3dd
commit
8120ed457c
1 changed files with 20 additions and 19 deletions
39
.github/PULL_REQUEST_TEMPLATE.md
vendored
39
.github/PULL_REQUEST_TEMPLATE.md
vendored
|
@ -1,32 +1,33 @@
|
||||||
<!--
|
<!--
|
||||||
Please describe your pull request:
|
|
||||||
- What does it implement / fix / improve?
|
|
||||||
- Is it related to an existing issue?
|
|
||||||
-->
|
|
||||||
|
|
||||||
<!--
|
Thank you for your interest in contributing!
|
||||||
After submitting your first pull request, you will automatically be asked to accept our Contributor License Agreement (CLA):
|
|
||||||
|
|
||||||
https://github.com/photoprism/photoprism/blob/develop/CONTRIBUTING.md#contributor-license-agreement-cla
|
Because we want to create the best possible product for our users, we have a set of criteria to ensure that all submissions are acceptable, see https://docs.photoprism.app/developer-guide/pull-requests/ for details.
|
||||||
|
|
||||||
|
(1) Please provide a concise description of your pull request.
|
||||||
|
|
||||||
|
- What does it implement / fix / improve? Why?
|
||||||
|
- Are the changes related to an existing issue?
|
||||||
|
|
||||||
|
(2) After you submit your first pull request, you will be asked to accept our CLA, see https://photoprism.app/cla.
|
||||||
|
|
||||||
|
(3) Finally, please confirm that the following criteria are met by replacing "[ ]" with "[x]" (also possible at a later time).
|
||||||
|
|
||||||
Because we want to create the best possible product for our users, we have a set of guidelines to ensure that all submissions are acceptable.
|
|
||||||
Please check the following items by replacing "[ ]" with "[x]".
|
|
||||||
You can also do this when viewing the pull request after it was created:
|
|
||||||
-->
|
-->
|
||||||
|
|
||||||
Acceptance Criteria:
|
Acceptance Criteria:
|
||||||
|
|
||||||
- [ ] **Features and enhancements are fully implemented** so that they can be released at any time without additional work
|
- [ ] Features and enhancements must be fully implemented so that they can be released at any time without additional work
|
||||||
- [ ] **Automated unit and/or acceptance tests have been added** to ensure the changes work as expected and to reduce repetitive manual work
|
- [ ] Automated unit and/or acceptance tests are mandatory to ensure the changes work as expected and to reduce repetitive manual work
|
||||||
- [ ] **User interface changes are fully responsive** and have been tested on all major browsers and various devices
|
- [ ] Frontend components must be responsive to work and look properly on phones, tablets, and desktop computers; you must have tested them on all major browsers and different devices
|
||||||
- [ ] Database-related changes are compatible with SQLite and MariaDB
|
- [ ] Documentation and translation updates should be provided if needed
|
||||||
- [ ] Translations have been / will be updated (specify if needed)
|
- [ ] In case you submit database-related changes, they must be tested and compatible with SQLite 3 and MariaDB 10.5.12+
|
||||||
- [ ] Documentation has been / will be updated (specify if needed)
|
|
||||||
- [ ] Contributor License Agreement (CLA) has been signed
|
|
||||||
|
|
||||||
<!--
|
<!--
|
||||||
Reviewing, testing and finally merging pull requests consumes significant resources on our side. Unless it's just a small fix, it may take several months.
|
|
||||||
|
|
||||||
Thanks for your patience :)
|
Since reviewing, testing and finally merging pull requests requires significant resources on our side, this can take several months if it's not just a small fix, especially if extensive testing is required to prevent bugs from getting into our stable version.
|
||||||
|
|
||||||
|
We thank you for your patience! :)
|
||||||
|
|
||||||
-->
|
-->
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue