ci: make yarn install --immutable explicit in CI workflows - #10389
ci: make yarn install --immutable explicit in CI workflows#10389dev-hari-prasad wants to merge 1 commit into
Conversation
Yarn's enableImmutableInstalls already defaults to true when CI is set in the environment. Passing --immutable explicitly makes the intent clear and self-documenting in all CI workflows that invoke Yarn.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (4)
Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review. WalkthroughThe JavaScript style, EPAS feature-test, PostgreSQL feature-test, and JavaScript test workflows now use ChangesCI install consistency
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to The workflows now fail rather than silently updating an inconsistent Yarn lockfile, improving CI dependency reproducibility. No current merge-readiness risk is identified. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Part of #10363.
Yarn's
enableImmutableInstallssetting already defaults totruewhenCIis present in the environment (such as in GitHub Actions).Passing
--immutableexplicitly toyarn installin our CI workflows makes the intended immutable lockfile enforcement self-documenting and transparent to future contributors and maintainers.Changes
--immutabletoyarn installacross the 4 GitHub Actions workflows:check-javascript-style.ymlrun-javascript-tests.ymlrun-feature-tests-epas.ymlrun-feature-tests-pg.ymlSummary by CodeRabbit