fix: remove redundant check and fix typo in pr-podman-push (#3361)

Signed-off-by: Frank Kong <[email protected]>

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED

Author: Zaperex

.github/workflows/pr-podman-push.yaml

@@ -43,7 +43,7 @@ jobs:
             echo "PR branch: $PR_BRANCH"
             echo "Artifact Name: container-image-pr-$PR_NUMBER-$SHORT_SHA"
 
-            echo "pr-number=$PR_NUMBER" >> $GITHUB_OUTPUT
+            echo "pr_number=$PR_NUMBER" >> $GITHUB_OUTPUT
             echo "short-sha=$SHORT_SHA" >> $GITHUB_OUTPUT
             echo "artifact-name=container-image-pr-$PR_NUMBER-$SHORT_SHA" >> $GITHUB_OUTPUT
           else
@@ -53,7 +53,7 @@ jobs:
       - name: Determine artifact name
         run: |
           # For workflow_run, extract from the event context
-          BUILD_ID="${{ steps.get-pr-info.outputs.pr-number }}"
+          BUILD_ID="${{ steps.get-pr-info.outputs.pr_number }}"
           SHORT_SHA="${{ github.event.workflow_run.head_sha }}"
           SHORT_SHA="${SHORT_SHA:0:8}"
 
@@ -140,7 +140,7 @@ jobs:
           echo "(either due to [skip-build] tag or no relevant changes with existing image)"
 
       - name: Comment the image pull link
-        if: ${{ steps.check-skip.outputs.is_skipped != 'true' && github.event_name == 'workflow_run' && steps.get-pr-info.outputs.pr_number }}
+        if: ${{ steps.check-skip.outputs.is_skipped != 'true' }}
         uses: actions/github-script@v7
         env:
           PUSHED_TAGS: ${{ steps.prepare.outputs.tags }}