revert(ci): revert datahub-wheel build changes (datahub-project#12747)

shirshanka · Mar 3, 2025 · 3268264 · 3268264
1 parent 0c229d5
commit 3268264
Showing 1 changed file with 3 additions and 13 deletions.
diff --git a/.github/workflows/python-build-pages.yml b/.github/workflows/python-build-pages.yml
@@ -8,7 +8,7 @@ on:
       - "metadata-ingestion/**"
       - "metadata-ingestion-modules/**"
       - "metadata-models/**"
-  pull_request_target:
+  pull_request:
     branches:
       - "**"
     paths:
@@ -38,11 +38,7 @@ jobs:
           distribution: "zulu"
           java-version: 17
       - uses: gradle/actions/setup-gradle@v4
-      - uses: actions/checkout@v4
-        # Note: not using acryldata/sane-checkout-action because this is a
-        # pull_request_target event, and hence requires `ref`.
-        with:
-          ref: ${{ github.event.pull_request.head.sha }}
+      - uses: acryldata/sane-checkout-action@v3
       - uses: actions/setup-python@v5
         with:
           python-version: "3.10"
@@ -58,17 +54,11 @@ jobs:
         env:
           GITHUB_TOKEN: ${{ github.token }}
       - name: Publish
-        # Cloudflare's pages-action does not support PRs from forks.
-        # See https://developers.cloudflare.com/pages/platform/known-issues/
-        # Based on the discussion here https://github.com/json-schema-org/website/issues/330
-        # we can use a direct upload to work around this limitation.
-        # See https://github.com/AdrianGonz97/refined-cf-pages-action#enabling-pr-previews-from-forks
-        uses: AdrianGonz97/refined-cf-pages-action@v1
+        uses: cloudflare/pages-action@v1
         with:
           apiToken: ${{ secrets.CLOUDFLARE_API_TOKEN }}
           accountId: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
           projectName: ${{ vars.CLOUDFLARE_WHEELS_PROJECT_NAME }}
-          deploymentName: ${{ vars.CLOUDFLARE_WHEELS_PROJECT_NAME }}
           workingDirectory: python-build
           directory: site
           gitHubToken: ${{ github.token }}