Closed flongford closed 4 years ago
Merging #337 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #337 +/- ##
=======================================
Coverage 98.40% 98.40%
=======================================
Files 60 60
Lines 1694 1694
Branches 172 172
=======================================
Hits 1667 1667
Misses 23 23
Partials 4 4
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 0c926ea...91f9cd5. Read the comment docs.
Should the PR review be in a specific format?
Good question - I don't think so, and in any case I don't think GitHub provides a way to contribute a template for a PR review, so it would just be a pain to have to copy + paste each time.
Description
This PR introduces a Markdown template that can be used in future PRs to the BDSS.
Including it in the
.github
directory will make it discoverable by GitHub: https://help.github.com/en/github/building-a-strong-community/creating-a-pull-request-template-for-your-repositoryAdditional reviewer instructions for developers are also provided
Changes
.github/pull_request_template.md
developer_guidelines.rst