Merge pull request #45 from Cysharp/release

ci: Improve consistency for nuget publish

Author: guitarrapc

.github/workflows/build-release.yaml

@@ -12,19 +12,31 @@ on:
         default: false
         type: boolean
 
+concurrency:
+  group: release
+  cancel-in-progress: false
+
 jobs:
+  validate-release:
+    permissions:
+      contents: read
+    uses: Cysharp/Actions/.github/workflows/validate-release.yaml@main
+    with:
+      tag: ${{ inputs.tag }}
+
   update-packagejson:
+    needs: [validate-release]
     permissions:
       actions: read
       contents: write
     uses: Cysharp/Actions/.github/workflows/update-packagejson.yaml@main
     with:
       file-path: ./RuntimeUnitTestToolkit/Assets/RuntimeUnitTestToolkit/package.json
-      tag: ${{ inputs.tag }}
+      tag: ${{ needs.validate-release.outputs.tag }}
       dry-run: ${{ inputs.dry-run }}
 
   build-unity:
-    needs: [update-packagejson]
+    needs: [validate-release, update-packagejson]
     strategy:
       matrix:
         unity: ["2022.3.39f1"]
@@ -55,7 +67,7 @@ jobs:
           UNITY_EMAIL: ${{ steps.op-load-secret.outputs.UNITY_EMAIL }}
           UNITY_PASSWORD: ${{ steps.op-load-secret.outputs.UNITY_PASSWORD }}
           UNITY_SERIAL: ${{ steps.op-load-secret.outputs.UNITY_SERIAL }}
-          UNITY_PACKAGE_VERSION: ${{ inputs.tag }}
+          UNITY_PACKAGE_VERSION: ${{ needs.validate-release.outputs.tag }}
         with:
           projectPath: RuntimeUnitTestToolkit
           unityVersion: ${{ matrix.unity }}
@@ -69,29 +81,27 @@ jobs:
       # Store artifacts.
       - uses: Cysharp/Actions/.github/actions/upload-artifact@main
         with:
-          name: RuntimeUnitTestToolkit.${{ inputs.tag }}.unitypackage
-          path: ./RuntimeUnitTestToolkit/RuntimeUnitTestToolkit.${{ inputs.tag }}.unitypackage
+          name: RuntimeUnitTestToolkit.${{ needs.validate-release.outputs.tag }}.unitypackage
+          path: ./RuntimeUnitTestToolkit/RuntimeUnitTestToolkit.${{ needs.validate-release.outputs.tag }}.unitypackage
           retention-days: 1
 
   # release
   create-release:
-    needs: [update-packagejson, build-unity]
+    needs: [validate-release, update-packagejson, build-unity]
     permissions:
       contents: write
-      id-token: write
     uses: Cysharp/Actions/.github/workflows/create-release.yaml@main
     with:
       commit-id: ${{ needs.update-packagejson.outputs.sha }}
       dry-run: ${{ inputs.dry-run }}
-      tag: ${{ inputs.tag }}
-      nuget-push: false
+      tag: ${{ needs.validate-release.outputs.tag }}
       release-upload: true
-      release-asset-path: ./RuntimeUnitTestToolkit.${{ inputs.tag }}.unitypackage
+      release-asset-path: ./RuntimeUnitTestToolkit.${{ needs.validate-release.outputs.tag }}.unitypackage
     secrets: inherit
 
   cleanup:
-    if: ${{ needs.update-packagejson.outputs.is-branch-created == 'true' }}
-    needs: [update-packagejson, build-unity]
+    if: ${{ always() && needs.update-packagejson.outputs.is-branch-created == 'true' }}
+    needs: [update-packagejson, create-release]
     permissions:
       contents: write
     uses: Cysharp/Actions/.github/workflows/clean-packagejson-branch.yaml@main