github: update PR template after CI/CD migration
-This PR has the objective of...
+This PR has the objective of <describe here>
## Changes
@@ -10,21 +10,25 @@ This PR has the objective of...
## Checklist
-Checklist for for pull requests including new code and/or changes to existing code...
+Use this checklist for assuring the quality of pull requests that include new code and or make changes to existing code.
-* [ ] Code
+* [ ] Source Code rules apply:
* [ ] Includes unit tests
* [ ] New functions have docstrings with examples that can be run with doctest
* [ ] New functions are included in API docs
* [ ] Docstrings include notes for any changes to API or behaviour
- * [ ] All changes documented in docs/changes.rst
-* [ ] Testing
- * [ ] \(Optional) Tested local against remote servers
- * [ ] Travis CI passes (unit tests run under Linux)
- * [ ] AppVeyor CI passes (unit tests run under Windows)
- * [ ] Unit test coverage has not decreased (see Coveralls)
-* [ ] Changes
- * [ ] \(Optional) Just a proof of concept
- * [ ] \(Optional) Work in progress
+ * [ ] All changes are documented in docs/changes.rst
+* [ ] Versioning and history tracking rules apply:
+ * [ ] Using atomic commits when possible
+ * [ ] Commits are reversible when possible
+ * [ ] There is no incomplete changes in the pull request
+ * [ ] There is no accidental garbage added in source code
+* [ ] Testing rules apply:
+ * [ ] Tested locally using `tox` / `pytest`
+ * [ ] Automated testing passes (see [CI](https://github.com/petl-developers/petl/actions))
+ * [ ] Unit test coverage has not decreased (see [Coveralls](https://coveralls.io/github/petl-developers/petl))
+* [ ] State of these changes is:
+ * [ ] Just a proof of concept
+ * [ ] Work in progress / Further changes needed
* [ ] Ready to review
* [ ] Ready to merge
