Skip to content

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

Merged
0x46616c6b merged 4 commits into
mainfrom
claude/youthful-curie-e9k9aq
Jun 30, 2026
Merged

fix(techdocs): publish only changed components on push#464
0x46616c6b merged 4 commits into
mainfrom
claude/youthful-curie-e9k9aq

Conversation

@flaxel

@flaxel flaxel commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

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

Changes

  • .github/workflows/template_techdocs_monorepo.yml: change runs-on from ubuntu-slim to ubuntu-24.04.

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.

@0x46616c6b 0x46616c6b left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we simply revert the usage of ubuntu-slim? I would like to see if this would be enough and omit the complex code change.

@flaxel

flaxel commented Jun 30, 2026

Copy link
Copy Markdown
Contributor Author

I thought it is a good improvement as delta is always faster and you don't need to build the same things every time but I will revert it and we will see. 🎉

flaxel and others added 2 commits June 30, 2026 08:54
The revert commit stripped the trailing newline, tripping yamllint's
new-line-at-end-of-file rule.

Co-authored-by: opencode <opencode@noreply.opencode.ai>
Co-authored-by: GitHub Copilot <copilot@noreply.github.com>
@flaxel
flaxel requested a review from 0x46616c6b June 30, 2026 06:58
@0x46616c6b
0x46616c6b merged commit f5e49d0 into main Jun 30, 2026
7 of 8 checks passed
@0x46616c6b
0x46616c6b deleted the claude/youthful-curie-e9k9aq branch June 30, 2026 07:12
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

5 participants