Skip to content

Skip docs sync pushes when nothing changed - #9

Merged
jpshackelford merged 1 commit into
mainfrom
docs-sync-skip-unchanged
Oct 3, 2026
Merged

jpshackelford merged 1 commit into
mainfrom
docs-sync-skip-unchanged

Conversation

@jpshackelford

@jpshackelford jpshackelford commented Oct 3, 2026 •

Copy link
Copy Markdown
Member

Why

The nightly Docs publish run re-renders every page from scratch and committed and force-pushed the result even when it matched what was already on the cookbook-sync branch. That re-ran the docs repository's CI and the Mintlify deployment on the sync PR every night for no change. docs_sync.py push now compares the rendered tree with the existing branch and only pushes when they differ. Preview branches get the same check, so re-running a preview on an unchanged PR reuses the deployment that already exists.

Validation

  • Tree comparison — against a scratch bare repository: no branch means push, an identical render means skip, and a changed render means push.
  • Live re-run on this PR — re-running Docs preview logged cookbook-preview/pr-9 already has these files; not pushing. The docs branch head was unchanged, and the run still reported the existing Mintlify preview.

This PR was drafted by an AI agent on behalf of the user.

@jpshackelford can click here to continue refining the PR

The nightly publish run re-renders from scratch; without this it force-pushed
an identical tree every night, re-running docs CI and the Mintlify deploy on the
sync PR for no change.

Co-authored-by: openhands <openhands@all-hands.dev>
@github-actions

github-actions Bot commented Oct 3, 2026

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py50100% 
cli.py65395%59–60, 100
client.py1051784%56, 109, 117, 150, 160–162, 173, 203, 206–208, 210, 217–219, 221
metrics.py80298%102, 182
v0.py48394%88, 104, 141
v1.py83989%86, 90, 94, 131, 135, 155, 178, 206, 214
TOTAL3863491% 

@github-actions

github-actions Bot commented Oct 3, 2026

Copy link
Copy Markdown

Docs preview

✅ Docs preview is ready: https://allhandsai-cookbook-preview-pr-9.mintlify.site

Page Change
/cookbook/conversation-tags added
/cookbook added

Docs PR: OpenHands/docs#887 (draft preview; never merged, closes with this PR).
Rendered from f00d868. After merge, a separate sync PR updates the docs.

@jpshackelford
jpshackelford marked this pull request as ready for review October 3, 2026 18:37
@jpshackelford
jpshackelford merged commit 83191ed into main Oct 3, 2026
11 checks passed
@jpshackelford
jpshackelford deleted the docs-sync-skip-unchanged branch October 3, 2026 18:37
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