[pull] latest from ag-grid:latest#737
Merged
Merged
Conversation
* Reduce warm CI time from ~15m to <10m Decouple init job from critical path — format_lint_build, docs, and calc_matrix no longer wait for init's 4.6m yarn install. On warm cache the RO restore in each job is sufficient (22s vs 256s). Init still runs in parallel to keep the cache warm for future runs. Skip link checking on PRs — CHECK_LINKS/CHECK_REDIRECTS only set on push events. Push-to-latest still catches broken links before staging deploy. Expected new critical path: detect-changes (0.3m) → docs (~5-6m) → report (0.8m) ≈ 7m * Run link checks on PRs with documentation changes Enable CHECK_LINKS/CHECK_REDIRECTS when detect-changes finds docs content changes (run_docs_ci == true). Code-only PRs still skip link checking for speed. * Merge calc_matrix into init, split lint from build 1. Merge calc_matrix into init: eliminates a redundant job with its own setup-nx overhead (~1m). Init already has Nx configured, so the matrix calculation adds negligible time. Matches the ag-charts pattern where init handles both cache warming and matrix computation. 2. Split format_lint_build into separate lint and build jobs: lint runs for all PRs (including drafts) giving fast feedback; build only runs when code changes are detected. Matches ag-charts' independent lint job pattern. test/e2e now depend on init (which includes matrix outputs). For the typical PR critical path (docs ~7m), this has no impact since init (~4m) + test (~2m) = ~6m finishes before docs.
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )