Merge pull request #264 from carpentries/fix-pr-receive

fix artifact name for PRs with changed workflows
carpentries · Mar 2, 2022 · 8ef6993 · 8ef6993
2 parents 2d507e5 + 5bc106c
commit 8ef6993
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 4 deletions.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,6 +1,6 @@
 Package: sandpaper
 Title: Create and Curate Carpentries Lessons
-Version: 0.3.3
+Version: 0.3.4
 Authors@R: c(
     person(given = "Zhian N.",
            family = "Kamvar",

diff --git a/NEWS.md b/NEWS.md
@@ -1,3 +1,11 @@
+# sandpaper 0.3.4
+
+CONTINUOUS INTEGRATION
+----------------------
+
+ * pr-receive.yaml has been updated to not report errors for workflow updates.
+   See https://github.com/carpentries/sandpaper/issues/263 for details
+
 # sandpaper 0.3.3
 
 BUG FIXES

diff --git a/inst/workflows/pr-receive.yaml b/inst/workflows/pr-receive.yaml
@@ -111,18 +111,18 @@ jobs:
       - name: "Record PR number"
         id: record
         run: |
-          echo ${{ github.event.number }} > ${{ github.workspace }}/pr
+          echo ${{ github.event.number }} > ${{ github.workspace }}/NR # 2022-03-02: artifact name fixed to be NR
       - name: "Upload PR number"
         id: upload
         uses: actions/upload-artifact@v2
         with:
           name: pr
-          path: ${{ github.workspace }}/pr
+          path: ${{ github.workspace }}/NR
       - name: "Comment on PR"
         id: comment-diff
         uses: carpentries/actions/comment-diff@main
         with:
-          pr: ${{ github.event.number }} 
+          pr: ${{ github.event.number }}
           body: |
             This pull request contains modified workflows and no preview will be created.