Skip to content

Commit 483afa5

Browse files
authored
Merge pull request #129 from jbusecke/patch-6
Potential follow up fix for #125
2 parents 28fde27 + 1393c0f commit 483afa5

File tree

1 file changed

+21
-13
lines changed

1 file changed

+21
-13
lines changed

.github/workflows/deploy-book.yaml

+21-13
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,8 @@ jobs:
3636
run:
3737
shell: bash -l {0}
3838
steps:
39+
- uses: actions/checkout@v4
40+
3941
- name: Download merged artifact
4042
if: inputs.is_preview != 'true'
4143
uses: actions/download-artifact@v4
@@ -59,27 +61,33 @@ jobs:
5961
unzip book.zip
6062
rm -f book.zip
6163
64+
- name: Debug
65+
run: pwd
66+
67+
- name: Debug
68+
run: ls -la book/_build/html
69+
6270
- name: Deploy to GitHub Pages
6371
uses: JamesIves/github-pages-deploy-action@v4
6472
if: |
6573
(github.ref == 'refs/heads/main' && inputs.cname == 'None')
6674
with:
67-
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
68-
BRANCH: gh-pages
69-
FOLDER: ${{ inputs.publish_dir }}
70-
CLEAN: true
71-
KEEP_HISTORY: false
72-
BASE_PATH: ${{ inputs.destination_dir }}
75+
token: ${{ secrets.GITHUB_TOKEN }}
76+
branch: gh-pages
77+
folder: ${{ inputs.publish_dir }}
78+
clean: true
79+
clean-exclude: "preview" # keep existing previews from other PRs
80+
target-folder: ${{ inputs.destination_dir }}
7381

7482
- name: Deploy to GitHub Pages with custom domain
7583
uses: JamesIves/github-pages-deploy-action@v4
7684
if: |
7785
(github.ref == 'refs/heads/main' && inputs.cname != 'None')
7886
with:
79-
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
80-
BRANCH: gh-pages
81-
FOLDER: ${{ inputs.publish_dir }}
82-
CLEAN: true
83-
KEEP_HISTORY: false
84-
BASE_PATH: ${{ inputs.destination_dir }}
85-
CNAME: ${{ inputs.cname }}
87+
token: ${{ secrets.GITHUB_TOKEN }}
88+
branch: gh-pages
89+
folder: ${{ inputs.publish_dir }}
90+
clean: true
91+
clean-exclude: "preview" # keep existing previews from other PRs
92+
target-folder: ${{ inputs.destination_dir }}
93+
CNAME: ${{ inputs.cname }} # how can we set this for the new action?

0 commit comments

Comments
 (0)