Merge pull request #419 from powercasgamer/fix/action

fix(action): (probably) fix preview action
PaperMC · Jun 16, 2024 · 442ddd0 · 442ddd0
2 parents d90f3b5 + c29e808
commit 442ddd0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 26 deletions.
diff --git a/.github/workflows/deploy-pr.yaml b/.github/workflows/deploy-pr.yaml
@@ -10,22 +10,12 @@ jobs:
       deployments: write
       pull-requests: write
     steps:
-      - name: "add comment"
-        if: github.event_name != 'push'
-        uses: mshick/add-pr-comment@v2
-        with:
-          message-id: "cloudflare-pages-preview"
-          message: |
-            ## Deploying with  [<img alt="Cloudflare Pages" width="16" src="https://user-images.githubusercontent.com/23264/106598434-9e719e00-654f-11eb-9e59-6167043cfa01.png">](https://pages.dev)  Cloudflare Pages
-
-            |  Latest commit: | ${{ github.sha }} |
-            |---------------------------------|------------------------|
-            | Status: | 🔨  Building... |
-            ---
       - name: "checkout"
         uses: "actions/checkout@v4"
         with:
           fetch-depth: 0
+          ref: ${{ github.event.pull_request.head.ref }}
+          repository: ${{ github.event.pull_request.head.repo.full_name }}
       - name: "setup pnpm"
         uses: "pnpm/action-setup@v4"
       - name: "setup node"
@@ -49,17 +39,5 @@ jobs:
           accountId: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
           projectName: "papermc-docs"
           deploymentName: "Preview"
+          gitHubToken: ${{ secrets.GITHUB_TOKEN }}
           directory: "build"
-      - name: "add comment"
-        uses: "mshick/add-pr-comment@v2"
-        with:
-          message-id: "cloudflare-pages-preview"
-          message: |
-            ## Deploying with  [<img alt="Cloudflare Pages" width="16" src="https://user-images.githubusercontent.com/23264/106598434-9e719e00-654f-11eb-9e59-6167043cfa01.png">](https://pages.dev)  Cloudflare Pages
-
-            |  Latest commit: | ${{ github.sha }} |
-            |---------------------------------|------------------------|
-            | Status: | ✅  Deploy successful! |
-            | Preview URL: | [${{ steps.cloudflare-publish.outputs.url }}](${{ steps.cloudflare-publish.outputs.url }}) |
-
-            [View logs](${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }})
diff --git a/config-specs/paper/paper-global.yml b/config-specs/paper/paper-global.yml
@@ -412,7 +412,7 @@ unsupported-settings:
     description: >-
       Whether the server should skip damage ticks when entities are blocking damage
       via a shield. While Vanilla does process a damage tick, this behavior was a long
-      standing bug in paper. Enabling this option means that the damage tick will be
+      standing bug in Paper. Enabling this option means that the damage tick will be
       skipped, which may result in rapid damage on shields due to the missing invulnerability.
   perform-username-validation:
     default: "true"