fix: added missing pr number input (#4)

hey-car · Mar 22, 2024 · d2f1486 · d2f1486
1 parent eec2e7b
commit d2f1486
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/action.yml b/action.yml
@@ -31,6 +31,9 @@ inputs:
     description: AWS ECR Repo Alpha image
     required: false
     default: 'true'
+  pr-number:
+    description: The current pull request number
+    required: true
 
 runs:
   using: 'docker'
@@ -43,3 +46,4 @@ runs:
     ECR_REPO_NAME: ${{ inputs.ecr-repo-name }}
     ECR_REPO_TAG: ${{ inputs.ecr-repo-tag }}
     USE_ALPHA_REGISTRY: ${{ inputs.use-alpha }}
+    PR_NUMBER: ${{ inputs.pr-number }}
diff --git a/scripts/script.sh b/scripts/script.sh
@@ -5,10 +5,11 @@
 
 REPO_ORG=${GITHUB_REPOSITORY_OWNER}
 REPO_NAME=$(echo "${GITHUB_REPOSITORY}" |cut -d "/" -f2)
-PR_NUMBER=${GITHUB_SHA}
+
 check_env_var "AWS_ACCOUNT_ID"
 check_env_var "ECR_REPO_NAME"
 check_env_var "ECR_REPO_TAG"
+check_env_var "PR_NUMBER"
 check_env_var "USE_ALPHA_REGISTRY"
 
 if [[ "$(check_bool "${USE_ALPHA_REGISTRY}")" ]]; then