phys2bids
phys2bids copied to clipboard
Add Main Reviewer checklist
Closes none, depends on #315
This is a draft because I'm not totally sure the workflow is right.
Proposed Changes
- Add a GH Action to create comment addressing the main reviewer when a PR gets assigned.
- Add the template to provide the main reviewer with a checklist of things to do before (and after) merging.
Codecov Report
Merging #316 into master will not change coverage. The diff coverage is
n/a.
@@ Coverage Diff @@
## master #316 +/- ##
=======================================
Coverage 94.81% 94.81%
=======================================
Files 9 9
Lines 848 848
=======================================
Hits 804 804
Misses 44 44
Wow, I'm so excited for this! Let me know if there's anything I can do to help with this.
This is a good idea! I will take a look at what you have so far, on Monday, and see if I have suggestions at this stage.
This PR was marked as a work in progress a while ago. Should I look at it soon or wait?
#315 was already merged, I think we can continue work on this PR.
The main problem with this PR is that I didn't test it yet. If someone that has a bit of free time wants to try this out in a fake repository, please do so and help out finishing it!
This PR might need updating, as well as more looking into how to make this work with our project board