Skip to content

Commit cd7b19c

Browse files
committed
Update pull request template and contribute.md
1 parent 1d52846 commit cd7b19c

File tree

2 files changed

+10
-13
lines changed

2 files changed

+10
-13
lines changed

CONTRIBUTE.md renamed to .github/CONTRIBUTE.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -21,5 +21,5 @@ Getting your pull request merged
2121
This is a volunteer project, so sometimes it may take a little while to merge
2222
your pull request.
2323

24-
There is a [guide with hints for getting your pull requests merged](https://github.com/m-kuhn/QGIS-Website/blob/qtcreator.rst/source/site/getinvolved/development/git.rst#pull-requests)
24+
There is a [guide with hints for getting your pull requests merged](https://www.qgis.org/en/site/getinvolved/development/qgisdevelopersguide/git.html#pull-requests)
2525
in the developers guide.

.github/PULL_REQUEST_TEMPLATE.md

+9-12
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,15 @@
11
## Description
2-
Include a few sentences describing the overall goals for this pull request.
2+
Include a few sentences describing the overall goals for this PR (pull request). If applicable also add screenshots.
33

44
## Checklist
55

6-
All checklist items should be addressed before a QGIS maintainer will review the request:
6+
> Reviewing is a process done by project maintainers, mostly on a volunteer basis. We try to keep the overhead as small as possible and appreciate if you help us to do so by completing the following items. Feel free to ask in a comment if you have troubles with any of them.
77
8-
- [ ] I've read the [QGIS Coding Standards](https://www.qgis.org/en/site/getinvolved/development/qgisdevelopersguide/codingstandards.html) and this PR complies with them
9-
- [ ] This PR passes all existing unit tests
10-
- [ ] New unit tests have been added for core changes
11-
- [ ] I've run the scripts/prepare-commit.sh script before each commit
128
- [ ] Commit messages are descriptive and explain the rationale for changes
13-
- [ ] Commits which fix bugs include "fixes #11111" in the commit message
14-
- [ ] Commits which add new features are tagged with \[FEATURE\] in the commit message
15-
- [ ] Commits which change the UI or existing user workflows are tagged with \[needs-docs\] in the commit message
16-
17-
18-
9+
- [ ] Commits which fix bugs include `fixes #11111` in the commit message next to the description
10+
- [ ] Commits which add new features are tagged with `[FEATURE]` in the commit message
11+
- [ ] Commits which change the UI or existing user workflows are tagged with `[needs-docs]` in the commit message and containt sufficient information in the commit message to be documented
12+
- [ ] I have read the [QGIS Coding Standards](https://www.qgis.org/en/site/getinvolved/development/qgisdevelopersguide/codingstandards.html) and this PR complies with them
13+
- [ ] This PR passes all existing unit tests (test results will be reported by travis-ci after opening this PR)
14+
- [ ] New unit tests have been added for core changes
15+
- [ ] I have run [the `scripts/prepare-commit.sh` script](https://github.com/qgis/QGIS/blob/master/.github/CONTRIBUTE.md#contributing-to-qgis) before each commit

0 commit comments

Comments
 (0)