Skip to content

fix(techdocs): publish only changed components on push#464

Open
flaxel wants to merge 2 commits into
mainfrom
claude/youthful-curie-e9k9aq
Open

fix(techdocs): publish only changed components on push#464
flaxel wants to merge 2 commits into
mainfrom
claude/youthful-curie-e9k9aq

Conversation

@flaxel

@flaxel flaxel commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

The monorepo TechDocs template republished every component on every push, generating and uploading docs sequentially (~13s each). As the number of components grew, the job stopped finishing in time and was reclaimed mid-run, surfacing as cancelled runs.

This change computes the push diff (github.event.before..sha) and only (re)publishes components whose directory changed. It falls back to a full publish when:

  • the diff cannot be determined (non-push events, missing/zero base commit, unfetchable base), or
  • a global file changes (the workflow itself or the repo-root mkdocs.yml).

It also switches this workflow to the normal Ubuntu runner (ubuntu-24.04) instead of ubuntu-slim.

Changes

  • .github/workflows/template_techdocs_monorepo.yml: add incremental publish logic to the "Generate and Publish TechDocs" step.
  • .github/workflows/template_techdocs_monorepo.yml: change runs-on from ubuntu-slim to ubuntu-24.04.

🤖 Generated with Claude Code


Generated by Claude Code

The monorepo TechDocs template republished every component on every push,
generating and uploading docs sequentially (~13s each). As the number of
components grew, the job stopped finishing within the ubuntu-slim runner's
runtime window and was reclaimed mid-run, surfacing as cancelled runs.

Compute the push diff (github.event.before..sha) and only (re)publish
components whose directory changed. Falls back to a full publish when the
diff cannot be determined (non-push events, missing/zero base commit,
unfetchable base) or when a global file changes (workflow or root mkdocs.yml).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017PAVX31ZoNgY3Zmbsa2JNi
@github-actions

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


0 out of 2 committers have signed the CLA.
@claude
@copilot
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@flaxel flaxel added enhancement New feature or request minor Pull requests with new features labels Jun 18, 2026
@flaxel

flaxel commented Jun 18, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Use the normal ubuntu runner for this action.

Co-authored-by: GitHub Copilot <copilot@noreply.github.com>

Copilot AI commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

@copilot Use the normal ubuntu runner for this action.

Updated in commit 02fc71e: the TechDocs monorepo workflow now uses the normal Ubuntu runner (ubuntu-24.04) instead of ubuntu-slim.

@flaxel flaxel marked this pull request as ready for review June 18, 2026 20:45
@flaxel flaxel requested a review from a team as a code owner June 18, 2026 20:45
@flaxel flaxel requested review from axdotl and soemo June 18, 2026 20:45
@axdotl

axdotl commented Jun 19, 2026

Copy link
Copy Markdown
Member

@flaxel Think a partial build is reasonable, but would also like to understand where the "the job stopped finishing in time" comes from. Where and how a time constraint/ timeout comes from?

@flaxel

flaxel commented Jun 19, 2026

Copy link
Copy Markdown
Contributor Author

@axdotl The time constraint is a hard configuration in the ubuntu slim image.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request minor Pull requests with new features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants