Skip to content

Revert the docs workflow added in #177 (the GPU docs job already builds and deploys) - #178

Merged
ChrisRackauckas merged 1 commit into
SciML:mainfrom
ChrisRackauckas-Claude:ci/drop-redundant-docs-workflow
Sep 22, 2026
Merged

ChrisRackauckas merged 1 commit into
SciML:mainfrom
ChrisRackauckas-Claude:ci/drop-redundant-docs-workflow

Conversation

@ChrisRackauckas-Claude

@ChrisRackauckas-Claude ChrisRackauckas-Claude commented Sep 19, 2026 •

Copy link
Copy Markdown
Member

Please ignore until reviewed by @ChrisRackauckas.

What changed and why

This reverts the workflow change from #177. That PR added .github/workflows/Documentation.yml (the reusable CPU workflow), but HighDimPDE already builds its documentation on a self-hosted GPU runner:

So main currently runs two identically named "Documentation" checks that both build and deploy the same site from different runners, and the CPU one cannot execute the package's GPU examples. #177 changed nothing else — its diff was .github/workflows/Documentation.yml only — so removing that file is a complete revert.

I checked every other docs workflow PR merged in the same sweep by scanning each repo's workflows at the PR's base commit for a docs build (docs/make.jl, Pkg.activate("docs"), a documentation.yml@ reusable call, or julia-docdeploy). HighDimPDE is the only one where the base already had a docs build; the other 16 added a first docs build and are not duplicates. A repo-wide scan of all 139 in-scope repos likewise finds no other repo with two docs-building workflows.

Verification

  • main's GPU docs job is green: run https://github.com/SciML/HighDimPDE.jl/actions/runs/35259116305 → Documentation=success.
  • grep -rl 'docs/make.jl' .github/workflows after this change matches only GPU.yml.
  • Removal cannot affect the published site, since the surviving job is the one that was already building and deploying it.

Not verified

  • I did not run the GPU docs job locally (no GPU here); the claim rests on the existing green run plus the fact that the removed workflow was added after it.

🤖 Generated with OpenCode (model: muse-spark-1.3-contributor-free); no public session URL (local session at /home/crackauc/sandbox/tmp_20260917_040834_91074).

HighDimPDE already builds its documentation on a self-hosted GPU runner: the
`gpu-docs` job (job name "Documentation") in `.github/workflows/GPU.yml`,
`runs-on: [self-hosted, gpu-v100]`, which is green on `main`. SciML#177 added a
second, CPU-based Documentation workflow, so `main` now starts two identically
named "Documentation" checks that both build and deploy the same site from
different runners, and the CPU one cannot execute the package's GPU examples.

Remove the duplicate and keep the existing GPU docs job as the single build.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Agent-Harness: OpenCode
Agent-Model: muse-spark-1.3-contributor-free
Agent-Session: local session at /home/crackauc/sandbox/tmp_20260917_040834_91074 (no public conversation URL)
@ChrisRackauckas-Claude ChrisRackauckas-Claude changed the title ci: drop the redundant docs workflow (GPU docs job already builds and deploys) Revert the docs workflow added in #177 (the GPU docs job already builds and deploys) Sep 19, 2026
@ChrisRackauckas
ChrisRackauckas marked this pull request as ready for review September 22, 2026 23:35
@ChrisRackauckas
ChrisRackauckas merged commit c09c2e3 into SciML:main Sep 22, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants