fix(github): workflow for log-metric (#2322)

* fix(github): workflow for log-metric

* fix(github): log metric premature

* chore(github): add comment for absolute path

Co-authored-by: 0618 <[email protected]>

Author: 0618

.github/workflows/publish-latest.yml

@@ -92,10 +92,11 @@ jobs:
     # Send data point 1 to metric PublishNPMFailure, if it's a failure
     runs-on: ubuntu-latest
     environment: ci
+    needs: publish
     if: ${{ failure() }}
     steps:
       - name: Log failure data point to metric PublishNPMFailure
-        uses: aws-amplify/amplify-ui/.github/actions/log-metric
+        uses: aws-amplify/amplify-ui/.github/actions/log-metric@main
         with:
           metric-name: PublishNPMFailure
           value: 1
@@ -106,10 +107,11 @@ jobs:
     # Send data point 0 to metric PublishNPMFailure, if  it's a success
     runs-on: ubuntu-latest
     environment: ci
+    needs: publish
     if: ${{ success() }}
     steps:
       - name: Log success data point to metric PublishNPMFailure
-        uses: aws-amplify/amplify-ui/.github/actions/log-metric
+        uses: aws-amplify/amplify-ui/.github/actions/log-metric@main
         with:
           metric-name: PublishNPMFailure
           value: 0

.github/workflows/run-and-test-builds.yml

@@ -100,10 +100,11 @@ jobs:
     # Send data point 1 to metricRunTimeTestsFailure, if it's a failure
     runs-on: ubuntu-latest
     environment: ci
+    needs: build
     if: ${{ failure() }}
     steps:
       - name: Log failure data point to metric RunTimeTestsFailure
-        uses: aws-amplify/amplify-ui/.github/actions/log-metric
+        uses: aws-amplify/amplify-ui/.github/actions/log-metric@main
         with:
           metric-name: RunTimeTestsFailure
           value: 1
@@ -114,10 +115,11 @@ jobs:
     # Send data point 0 to metric RunTimeTestsFailure, if  it's a success
     runs-on: ubuntu-latest
     environment: ci
+    needs: build
     if: ${{ success() }}
     steps:
       - name: Log success data point to metric RunTimeTestsFailure
-        uses: aws-amplify/amplify-ui/.github/actions/log-metric
+        uses: aws-amplify/amplify-ui/.github/actions/log-metric@main
         with:
           metric-name: RunTimeTestsFailure
           value: 0

.github/workflows/test-deploy-main.yml

@@ -172,10 +172,14 @@ jobs:
     # Send data point 1 to metric PublishNextFailure, if  it's a failure
     runs-on: ubuntu-latest
     environment: ci
+    needs: build-test
     if: ${{ failure() }}
     steps:
       - name: Log failure data point to metric PublishNextFailure
-        uses: aws-amplify/amplify-ui/.github/actions/log-metric
+        # using absolute path here to avoid checking out the repo. Same as other absolute paths in this repo.
+        # Pattern: `{owner}/{repo}/.github/workflows/{filename}@{ref}`.
+        # more: https://docs.github.com/en/actions/using-workflows/reusing-workflows#calling-a-reusable-workflow
+        uses: aws-amplify/amplify-ui/.github/actions/log-metric@main
         with:
           metric-name: PublishNextFailure
           value: 1
@@ -186,10 +190,11 @@ jobs:
     # Send data point 0 to metric PublishNextFailure, if  it's a success
     runs-on: ubuntu-latest
     environment: ci
+    needs: build-test
     if: ${{ success() }}
     steps:
       - name: Log success data point to metric PublishNextFailure
-        uses: aws-amplify/amplify-ui/.github/actions/log-metric
+        uses: aws-amplify/amplify-ui/.github/actions/log-metric@main
         with:
           metric-name: PublishNextFailure
           value: 0