There seem to be some common misconceptions about how to use the pull
request template such as:
- The `Fixes` field going before the pull request description.
- The `Testing` field being used to describe how the fix was tested
locally.
This change attmepts to make the meaning of the fields of the template a
little bit more explicit to avoid confusion.
Testing: This is just updating the pull request template so no tests
necessary.
Signed-off-by: Martin Robinson <mrobinson@fastmail.fm>
Co-authored-by: Martin Robinson <mrobinson@fastmail.fm>
The idea here is that we can use the pull request description as the
commit message, so that the git history is cleaner. This requires:
1. Removing comments from the message, as GitHub doesn't remove them
from the pull request descriptions when committing.
2. Remove the checklist, which is now verified by the CI instead, so
is less important.
3. Ask the contributor to remove the lines expicitly.
Signed-off-by: Martin Robinson <mrobinson@igalia.com>
