Skip to content

Fix API docs overlay workflow checkout#1669

Draft
llewellyn-sl wants to merge 4 commits into
masterfrom
codex/isolate-api-docs-version-checkout
Draft

Fix API docs overlay workflow checkout#1669
llewellyn-sl wants to merge 4 commits into
masterfrom
codex/isolate-api-docs-version-checkout

Conversation

@llewellyn-sl

Copy link
Copy Markdown
Contributor

Summary

  • isolate the sparse API version-check checkout under docs-version-check so the main checkout starts from a normal workspace
  • keep the duplicate-version check pointed at the sparse checkout path
  • pass the pinned Speakeasy version to the installer process so it does not drift to latest

Root cause

PR #1119 introduced a sparse checkout for the early version check and then reused the same workspace for the full checkout. The later workflow step expected .claude/skills/api-overlay-generator to exist in the root checkout, but the failed July run showed that path was absent from the runner worktree even though it existed in the commit. The later skill rename PR updated paths correctly; it changed the error path but likely did not introduce the underlying checkout issue.

Validation

  • git diff --check -- .github/workflows/generate-openapi-overlays.yml
  • pre-commit during commit: check yaml passed
  • shell sanity check confirmed VERSION reaches sh for the Speakeasy installer

Note: actionlint still reports pre-existing shellcheck warnings in unrelated workflow steps.

@netlify

netlify Bot commented Jul 20, 2026

Copy link
Copy Markdown

Deploy Preview for seqera-docs ready!

Name Link
🔨 Latest commit 2e17fc2
🔍 Latest deploy log https://app.netlify.com/projects/seqera-docs/deploys/6a6081333c7b2f00082c25e7
😎 Deploy Preview https://deploy-preview-1669--seqera-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@llewellyn-sl llewellyn-sl self-assigned this Jul 22, 2026
@llewellyn-sl
llewellyn-sl marked this pull request as ready for review July 22, 2026 08:03
@llewellyn-sl llewellyn-sl added the do not merge Do not merge until this label is removed label Jul 22, 2026
@llewellyn-sl
llewellyn-sl marked this pull request as draft July 22, 2026 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Do not merge until this label is removed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant