Merge pull request #92 from MetOffice/coverage-report-comments

Make PR coverage reports edit last comment
MetOffice · Apr 24, 2023 · c1c06f6 · c1c06f6
2 parents 0b1cb64 + 7de4b39
commit c1c06f6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/.github/workflows/pr_ci.yml b/.github/workflows/pr_ci.yml
@@ -53,7 +53,10 @@ jobs:
           badge_options="$(coverage json --fail-under=0 -qo - | jq -r .totals.percent_covered_display)%25-blue?style=for-the-badge"
           echo "[![Coverage](https://img.shields.io/badge/coverage-${badge_options})](${report_url})" >> ${{ runner.temp }}/cov-report.md
           echo "[Code coverage report](${report_url})" >> ${{ runner.temp }}/cov-report.md
-          gh pr comment ${{ github.head_ref }} --body-file ${{ runner.temp }}/cov-report.md
+          # Edit last comment if it exists, else create new one.
+          if ! gh pr comment --edit-last ${{ github.head_ref }} --body-file ${{ runner.temp }}/cov-report.md ; then
+            gh pr comment ${{ github.head_ref }} --body-file ${{ runner.temp }}/cov-report.md
+          fi
 
   pre-commit:
     runs-on: ubuntu-latest

diff --git a/requirements/environment.yml b/requirements/environment.yml
@@ -17,7 +17,6 @@ dependencies:
   - pip
 
   # Development dependencies
-  - pip
   - pre-commit
   - black
-Original file line number
+Diff line change
@@ Expand Up / @@ -17,7 +17,6 @@ dependencies: @@
       - pip
       # Development dependencies
-      - pip
       - pre-commit
       - black
@@ Expand Down @@