Merge pull request #4 from vincejv/fix-workflow-for-main

Update workflows
vincejv · Oct 14, 2022 · 2e12212 · 2e12212
2 parents 8861199 + edf2c51
commit 2e12212
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 4 deletions.
diff --git a/.github/workflows/package-release-dev.yml b/.github/workflows/package-release-dev.yml
@@ -9,8 +9,12 @@ jobs:
   deploy:
     name: Release artifact
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      packages: write
     steps:
-      - uses: actions/checkout@v3
+      - name: Checkout
+        uses: actions/checkout@v3
         with:
           fetch-depth: 0  # Shallow clones should be disabled for a better relevancy of analysis
       - name: Set up JDK 17

diff --git a/.github/workflows/package-release-main.yml b/.github/workflows/package-release-main.yml
@@ -22,13 +22,22 @@ jobs:
           tag: ${{ steps.tag_version.outputs.new_tag }}
           name: Release ${{ steps.tag_version.outputs.new_tag }}
           body: ${{ steps.tag_version.outputs.changelog }}
+      - name: Prepare release
+        id: prep_release
+        run: |
+          echo "latest_tag=steps.tag_version.outputs.new_tag" >> $GITHUB_ENV
   deploy:
     name: Release artifact
+    needs: versioning
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      packages: write
     steps:
-      - uses: actions/checkout@v3
+      - name: Checkout
+        uses: actions/checkout@v3
         with:
-          fetch-depth: 0  # Shallow clones should be disabled for a better relevancy of analysis
+          ref: '${{ env.latest_tag }}'
       - name: Set up JDK 17
         uses: actions/setup-java@v3
         with:

diff --git a/.github/workflows/sonarcloud.yml b/.github/workflows/sonarcloud.yml
@@ -13,7 +13,8 @@ jobs:
     name: Build
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - name: Checkout
+        uses: actions/checkout@v3
         with:
           fetch-depth: 0  # Shallow clones should be disabled for a better relevancy of analysis
       - name: Set up JDK 17