[build] fix parallelization for documentation release#17420
Conversation
Review Summary by QodoFix documentation release parallelization with concurrency control
WalkthroughsDescription• Remove max-parallel limit to allow docs jobs to run concurrently • Add concurrency control to prevent race conditions during commits • Simplify git push retry logic with shell conditional operator • Consolidate git setup and commit steps into single operation File Changes1. .github/workflows/release.yml
|
Code Review by Qodo
1. No rebase before gh-pages push
|
There was a problem hiding this comment.
Pull request overview
This PR adjusts the release documentation update workflow to allow per-language documentation generation to run in parallel while ensuring gh-pages writes are serialized to avoid push races.
Changes:
- Remove
max-parallel: 1from the release workflow docs matrix so documentation updates can run in parallel. - Add a
concurrencygroup to the reusable docs workflow’s commit job to serialize pushes togh-pages. - Simplify the commit step by removing the pull/retry loop and pushing once under the concurrency guard.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/update-documentation.yml | Serializes the gh-pages commit/push step via job-level concurrency and simplifies the commit logic. |
| .github/workflows/release.yml | Enables parallel execution of the per-language documentation update matrix. |
💥 What does this PR do?
Allows docs to run in parallel but makes the commits concurrent so we can remove race condition protection.
🤖 AI assistance
🔄 Types of changes