Repository navigation
Run independent GitHub Actions steps concurrently - #10497
Open
PascalSenn wants to merge 1 commit into
Open
PascalSenn wants to merge 1 commit into
PascalSenn wants to merge 1 commit into
Conversation
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The concurrency barriers preserve dependencies, and native CI confirms the new parallel syntax executes correctly.
Review effort: Balanced
Findings: None
What changed in this PR
Runs independent GitHub Actions steps concurrently to reduce CI and deployment duration.
Changes:
- Parallelizes website checks and artifact uploads.
- Installs Chromium in the background before deployment builds.
- Exercises website CI when
ci.ymlchanges.
| File | Description |
|---|---|
.github/workflows/ci.yml |
Parallelizes website checks and artifact uploads. |
.github/workflows/coverage.yml |
Uploads test and coverage artifacts concurrently. |
.github/workflows/publish-website.yml |
Installs Chromium asynchronously and waits before building. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Run independent work concurrently using the new GitHub Actions step syntax.
parallelgroup before generating assets and building.background: trueduring website deployment preparation, then explicitlywaitbefore the build, which uses Chromium to render Mermaid diagrams.ci.ymlchanges so workflow edits exercise this job.In a recent successful CI run, Chromium installation took 32 seconds, formatting 22 seconds and linting 29 seconds sequentially. This removes roughly 51 seconds of sequential work from that group before accounting for resource contention. Artifact uploads also overlap across the test matrix. All existing commands, pinned actions, artifact names, conditions and test failure handling are retained.
Validation: YAML parsing, synchronization and command preservation checks passed. actionlint v1.7.12 passed on equivalent sequential workflows; it does not yet recognize the new concurrency keywords. In the PR CI run, the three website preparation steps started together and completed successfully in a 42-second group. The GreenDonut.Tests test result and coverage uploads also started together and both succeeded. The website build started only after the group completed. Deployment and main branch coverage changes require their next normal run for execution validation.