Browse Source

Merge pull request #434 from AlgebraicJulia/checklist

BUILD: Fix error in github actions interpreting |
master
Micah Halter 2 weeks ago
committed by GitHub
parent
commit
110c55af02
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
  1. 43
      .github/workflows/reviewchecklist.yml

43
.github/workflows/reviewchecklist.yml

@ -1,7 +1,7 @@
name: Review Checklist
on:
pull_request_target:
pull_request:
types: [review_requested]
jobs:
@ -14,24 +14,23 @@ jobs:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
check_for_duplicate_msg: true
msg: |-
### Review Checklist
Does this PR follow the [development guidelines](https://github.com/AlgebraicJulia/Catlab.jl/wiki/Guidelines-for-PR-writers-and-reviewers)? Following is a *partial* checklist:
**Tests**
- [ ] New features and bug fixes have unit tests
- [ ] New modules have tests that are ultimately called by the test runner (`test/runtests.jl`)
- [ ] Existing tests have not been deleted
- [ ] Code coverage >= 90% or reduction justified in PR
**Documentation**
- [ ] All exported functions, types, and constants have docstrings, written in complete sentences
- [ ] Citations are given for any constructions, algorithms, or code drawn from external sources
**Other**
- [ ] Style guidelines are followed, including indent width 2
- [ ] Changes breaking backwards compatibility have been approved
msg: "### Review Checklist\n
\n
Does this PR follow the [development guidelines](https://github.com/AlgebraicJulia/Catlab.jl/wiki/Guidelines-for-PR-writers-and-reviewers)? Following is a *partial* checklist:\n
\n
**Tests**\n
\n
- [ ] New features and bug fixes have unit tests\n
- [ ] New modules have tests that are ultimately called by the test runner (`test/runtests.jl`)\n
- [ ] Existing tests have not been deleted\n
- [ ] Code coverage >= 90% or reduction justified in PR\n
\n
**Documentation**\n
\n
- [ ] All exported functions, types, and constants have docstrings, written in complete sentences\n
- [ ] Citations are given for any constructions, algorithms, or code drawn from external sources\n
\n
**Other**\n
\n
- [ ] Style guidelines are followed, including indent width 2\n
- [ ] Changes breaking backwards compatibility have been approved"
Loading…
Cancel
Save