ci: test both push and pull_request events

I think we have a bug with pull_request events so...

action.yml

@@ -91,10 +91,11 @@ runs:
       if: inputs.comment-on-pr == 'true' && steps.pr-number.outputs.pr-number != ''
       env:
         ARTIFACT_NAME: ${{ inputs.artifact-name }}
-        PR_ID: ${{ steps.pr-number.outputs.pr-number }}
+        BASE_BRANCH: ${{ inputs.base-branch }}
         COMMENT_ID: ${{ steps.find-comment.outputs.comment-id }}
         DO_COMPARISON: ${{ inputs.do-comparison }}
-        BASE_BRANCH: ${{ inputs.base-branch }}
+        JOB_NAME: ${{ inputs.job-name }}
+        PR_ID: ${{ steps.pr-number.outputs.pr-number }}
       run: |
         . "$GITHUB_ACTION_PATH/scripts/utils.sh"
 

scripts/retrieve-old-report.sh

@@ -41,10 +41,8 @@ has_report() {
 base_branch=${BASE_BRANCH:-$(default_branch)}
 
 if in_private_repo; then
-  warn 'Detected that this is a private repo, retrieving the old report will probably fail'
-fi
-
-if ! in_private_repo && [ "$JOB_NAME" ]; then
+  warn 'Detected that this is a private repo cannot retrieve old report'
+elif [ "$JOB_NAME" ]; then
   url=$(base_report_url "$base_branch")
 
   log "Found previous run at: $url"
@@ -60,6 +58,8 @@ if ! in_private_repo && [ "$JOB_NAME" ]; then
     exit 0
   fi
   error "Failed to find previous report, expected at: $report_url"
+else
+  panic 'job-name is missing, therefore we cannot find the previous report'
 fi
 
 warn "Couldn't retrieve old report: