PECOBLR-2023 Skip assembly jar packaging tests on fork PRs due to missing secrets (#1252)

The "Check Uber Jar Packaging" and "Check Thin Jar Packaging" steps in
the prCheck workflow rely on repository secrets (JDBC_PAT_TEST_TOKEN,
DATABRICKS_HOST, etc.) that are not available to workflows triggered by
pull requests from forks. This is a GitHub security restriction by
design.

These steps now include an if: github.event_name == 'push' condition so
they only run on push to main (i.e., after merge). They will be skipped
on PR events where secrets may be unavailable.

This does not reduce test coverage — the assembly packaging tests still
run post-merge on every push to main. All other PR checks (unit tests,
formatting, build) continue to run on PRs as before.

NO_CHANGELOG=true

Author: tejassp-db

.github/workflows/prCheck.yml

@@ -204,7 +204,9 @@ jobs:
         shell: bash
         run: mvn -B -pl jdbc-core,assembly-uber,assembly-thin install -DskipTests -Dmaven.javadoc.skip=true -Dmaven.source.skip=true -Ddependency-check.skip=true
 
+      # Only run on push to main; secrets are not available for fork PRs
       - name: Check Uber Jar Packaging
+        if: github.event_name == 'push'
         shell: bash
         run: mvn -pl test-assembly-uber test
         env:
@@ -213,7 +215,9 @@ jobs:
           DATABRICKS_USER: ${{ secrets.DATABRICKS_USER }}
           DATABRICKS_TOKEN: ${{ secrets.JDBC_PAT_TEST_TOKEN }}
 
+      # Only run on push to main; secrets are not available for fork PRs
       - name: Check Thin Jar Packaging
+        if: github.event_name == 'push'
         shell: bash
         run:  mvn -pl test-assembly-thin test
         env: