move comment to PR target

Former-commit-id: 3e90f5e

Author: scottgigante-immunai

.github/pull_request_template.md

@@ -15,9 +15,6 @@
 * [ ] This submission was written on a forked copy of openproblems
 * [ ] Nextflow test pipeline is passing on this base branch of this pull
   request (include link to passed test on NF Tower found in GitHub Actions summary: )
-* [ ] If this pull request is not ready for review (including passing the Nextflow test
-  pipeline), I will open this PR as a draft (click on the down arrow next to the
-  "Create Pull Request" button)
 
 ### Submission guidelines
 
@@ -31,11 +28,6 @@
 This PR will be evaluated on the basis of the following checks:
 
 * [ ] The task addresses a valid open problem in single-cell analysis
-* [ ] The latest version of master is merged and tested
-* [ ] The methods/metrics are imported to `__init__.py` and were tested in the pipeline
-* [ ] Method and metric decorators are annotated with paper title, year, author, code
-  version, and date
-* [ ] The README gives an outline of the methods, metrics and datasets in the folder
 * [ ] The README provides a satisfactory task explanation (for new tasks)
 * [ ] The sample test data is appropriate to test implementation of all methods and
   metrics (for new tasks)

.github/workflows/pre-commit.yml

@@ -19,11 +19,7 @@ jobs:
       options: --user root
 
     if: >-
-      !endsWith(github.event.head_commit.message, '# ci skip') &&
-      (
-        startsWith(github.ref, 'refs/heads') ||
-        github.event.pull_request.draft == false
-      )
+      !endsWith(github.event.head_commit.message, '# ci skip')
 
     steps:
 

.github/workflows/run_tests.yml

@@ -6,8 +6,9 @@ on:
       - '*'
     branches:
       - '**'
-  pull_request:
-    types: [opened, synchronize, reopened, ready_for_review]
+  pull_request_review:
+    types:
+      - 'submitted'
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}
@@ -21,10 +22,9 @@ jobs:
       !endsWith(github.event.head_commit.message, '# ci skip') &&
       !startsWith(github.ref, 'refs/heads/test_process') &&
       (
-        github.event_name != 'pull_request' ||
+        (github.event_name != 'pull_request_review') ||
         (
-          github.event_name == 'pull_request' &&
-          github.event.pull_request.draft == false &&
+          github.event.review.state == 'approved' &&
           github.event.pull_request.head.repo.owner.id == github.event.pull_request.base.repo.owner.id
         )
       )
@@ -147,12 +147,8 @@ jobs:
       !startsWith(github.ref, 'refs/heads/test_process') &&
       !startsWith(github.ref, 'refs/heads/test_website') &&
       (
-        github.event_name != 'pull_request' ||
-        (
-          github.event_name == 'pull_request' &&
-          github.event.pull_request.draft == false &&
-          github.actor != 'dependabot[bot]'
-        )
+        github.event_name != 'pull_request_review' ||
+        github.event.review.state == 'approved'
       )
 
     outputs:
@@ -374,7 +370,7 @@ jobs:
       always() &&
       !endsWith(github.event.head_commit.message, '# ci skip') &&
       needs.build_images.result == 'success' &&
-      github.event_name != 'pull_request' &&
+      github.event_name != 'pull_request_review' &&
       (
         needs.run_tester.result == 'success' ||
         needs.run_tester.result == 'skipped'