Commit 233bcb6
committed
Remove the redundant broken gh-pages deploy job
The repo has two workflows publishing to the same gh-pages branch:
main-docs.yml JamesIves/github-pages-deploy-action + the automatic
GITHUB_TOKEN -> works
documentation.yml webfactory/ssh-agent + secrets.GH_PAGES_DEPLOY
-> fails
The second has been failing on every push to main with
git@github.com: Permission denied (publickey)
because the GH_PAGES_DEPLOY key is no longer valid. It has gone unnoticed
because the site does still deploy - the other workflow does it, and needs
no configured secret.
Rather than rotate the key and end up with two workflows racing to publish
the same content, this deletes the gh-release job. documentation.yml
becomes PR validation only; main-docs.yml remains the single deploy path.
GH_PAGES_DEPLOY is now referenced nowhere and the repo secret can be
deleted.
This also removes a hardcoded git identity pointing at a former
contributor's personal email address.
Other fixes while in these files:
- main-docs.yml triggered on "push" with no branch filter, and its Deploy
step had no condition, so a push of any branch to this repo would
publish that branch's docs over the live site. Now scoped to main.
- Dropped its unused "python-version: [3.6]" matrix. The job only runs
yarn and npm; the value did nothing but label the check "(3.6)".
- Pinned Node 20 explicitly in both workflows. Docusaurus 3 requires
>= 20 and main-docs.yml was relying on the runner default, which is the
same implicit dependency that left documentation.yml stuck on Node 14.
- Switched main-docs.yml to "yarn install --frozen-lockfile", matching
documentation.yml, so the deploy builds what the lockfile pins.
- Dropped "sudo apt-get install -y yarn"; yarn is preinstalled on
ubuntu-latest and the apt package is a different tool.
- checkout/setup-node v1 and v2 -> v4, clearing the deprecation warnings.
The deploy action itself is left at JamesIves v3 deliberately. v4 renames
its inputs, and this is the only working deploy path, so it is not
something to change in the same PR that touches everything around it.
Verified locally on Node 20: "yarn install --frozen-lockfile" succeeds
against the committed lockfile and the build produces 50 pages.1 parent 76beaba commit 233bcb6
2 files changed
Lines changed: 23 additions & 43 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
3 | 6 | | |
4 | 7 | | |
5 | 8 | | |
6 | | - | |
7 | | - | |
8 | 9 | | |
9 | 10 | | |
10 | 11 | | |
11 | | - | |
12 | 12 | | |
13 | 13 | | |
14 | | - | |
15 | | - | |
| 14 | + | |
| 15 | + | |
16 | 16 | | |
| 17 | + | |
17 | 18 | | |
18 | 19 | | |
19 | 20 | | |
| |||
26 | 27 | | |
27 | 28 | | |
28 | 29 | | |
29 | | - | |
30 | | - | |
31 | | - | |
32 | | - | |
33 | | - | |
34 | | - | |
35 | | - | |
36 | | - | |
37 | | - | |
38 | | - | |
39 | | - | |
40 | | - | |
41 | | - | |
42 | | - | |
43 | | - | |
44 | | - | |
45 | | - | |
46 | | - | |
47 | | - | |
48 | | - | |
49 | | - | |
50 | | - | |
51 | | - | |
52 | | - | |
53 | | - | |
54 | | - | |
55 | | - | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
2 | 9 | | |
3 | 10 | | |
4 | 11 | | |
5 | 12 | | |
6 | | - | |
7 | | - | |
8 | | - | |
9 | 13 | | |
10 | 14 | | |
11 | | - | |
12 | | - | |
13 | | - | |
14 | | - | |
15 | | - | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
16 | 22 | | |
17 | 23 | | |
18 | 24 | | |
19 | | - | |
20 | | - | |
| 25 | + | |
| 26 | + | |
21 | 27 | | |
22 | 28 | | |
23 | 29 | | |
| |||
0 commit comments