Skip to content

fix(ci): avoid rebuilding docs in verify-generated-files - #2739

Merged
nicoloboschi merged 1 commit into
vectorize-io:mainfrom
Sanderhoff-alt:ci/speed-up-generated-files-check
Jul 20, 2026
Merged

fix(ci): avoid rebuilding docs in verify-generated-files#2739
nicoloboschi merged 1 commit into
vectorize-io:mainfrom
Sanderhoff-alt:ci/speed-up-generated-files-check

Conversation

@Sanderhoff-alt

Copy link
Copy Markdown
Contributor

Summary

  • Run the existing build-docs job for every PR so the production documentation build remains an unconditional check.
  • Generate OpenAPI directly in verify-generated-files instead of calling the wrapper that also rebuilds the Docusaurus site.
  • Preserve the existing generated-client, Rust release build, lint, and generated-file diff checks.

Impact

This removes a duplicate Docusaurus production build from the serialized verify-generated-files job while keeping the same documentation build coverage in the existing parallel build-docs job. Based on recent CI timings, verify-generated-files should drop from about 6:49 to about 3:38, making build-docs the longest job at about 4:03.

Validation

  • ./scripts/hooks/lint.sh
  • YAML parse check
  • git diff --check

@Sanderhoff-alt
Sanderhoff-alt force-pushed the ci/speed-up-generated-files-check branch from 25f9c9d to 73d218d Compare July 16, 2026 10:03
@Sanderhoff-alt Sanderhoff-alt changed the title ci: avoid duplicate docs build in generated-file check ci: avoid rebuilding docs in verify-generated-files Jul 16, 2026
Run the existing build-docs job for every PR so the production docs
build remains an unconditional check.

Generate OpenAPI directly in verify-generated-files to avoid rebuilding
the Docusaurus site serially in that job.
@Sanderhoff-alt
Sanderhoff-alt force-pushed the ci/speed-up-generated-files-check branch from 73d218d to c73c80e Compare July 16, 2026 10:04
@Sanderhoff-alt Sanderhoff-alt changed the title ci: avoid rebuilding docs in verify-generated-files fix(ci): avoid rebuilding docs in verify-generated-files Jul 16, 2026
@nicoloboschi
nicoloboschi merged commit dddd571 into vectorize-io:main Jul 20, 2026
87 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.

2 participants