Skip to content

Dedupe CI build/deploy and refresh README - #4

Merged
dannycab merged 1 commit into
mainfrom
planning
Sep 4, 2026
Merged

Dedupe CI build/deploy and refresh README#4
dannycab merged 1 commit into
mainfrom
planning

Conversation

@dannycab

@dannycab dannycab commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Deploy job was rebuilding slides and Hugo from scratch after the build job already did it, doubling CI time on every push/PR. Deploy now downloads the build job's public/ artifact instead of rebuilding.
  • Added an npm cache so npx @marp-team/marp-cli doesn't refetch from the registry on every run.
  • Refreshed README.md: documents the MARP slides workflow (previously missing entirely), fixes stale references to a nonexistent content/readings/ directory, corrects the file structure section, and updates the publishing section to match the deduped build/deploy flow.

Test plan

  • Confirm the build-deploy.yml workflow run on this PR succeeds and completes faster than a prior baseline run
  • Confirm deploy job only runs on push to main / manual dispatch and successfully publishes using the downloaded artifact

Deploy job was rebuilding slides and Hugo from scratch after the build
job had already done it, doubling CI time on every push. Deploy now
just downloads the build job's public/ artifact and publishes it.

Also refreshes README.md to describe the MARP slides workflow, fix
stale references to a nonexistent content/readings/ directory, and
document the deduped publish flow.
@dannycab
dannycab merged commit aa4cae6 into main Sep 4, 2026
2 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.

1 participant