Skip to content

CI: replace bare cargo install with cached baptiste0928/cargo-install#5180

Merged
jedel1043 merged 1 commit intoboa-dev:mainfrom
RishavTiwari25:ci/cache-cargo-install
Mar 20, 2026
Merged

CI: replace bare cargo install with cached baptiste0928/cargo-install#5180
jedel1043 merged 1 commit intoboa-dev:mainfrom
RishavTiwari25:ci/cache-cargo-install

Conversation

@RishavTiwari25
Copy link
Copy Markdown

This Pull Request fixes/closes #5179 .

It changes the following:

  • Replaced bare cargo install cargo-workspaces in the clippy, build-run-examples (rust.yml), and publish (release.yml) jobs with the cached baptiste0928/cargo-install action
  • Replaced bare cargo install --locked cargo-insta in the tests job (rust.yml) with baptiste0928/cargo-install (with locked: true)
  • Replaced bare cargo install cargo-fuzz in the fuzzing job (rust.yml) with baptiste0928/cargo-install

All 5 invocations now use baptiste0928/cargo-install@f204293d9709061b7bc1756fec3ec4e2cd57dec0 (v3.4.0), matching the pattern already used for cargo-tarpaulin, cross, and wasm-pack in the same repo. This caches installed binaries across CI runs, saving ~2–5 minutes per affected job on cache hits.

@github-actions github-actions bot added Waiting On Review Waiting on reviews from the maintainers C-Builtins PRs and Issues related to builtins/intrinsics C-Actions Pull requests that update Github Actions code and removed Waiting On Review Waiting on reviews from the maintainers labels Mar 20, 2026
@github-actions github-actions bot added this to the v1.0.0 milestone Mar 20, 2026
@github-actions
Copy link
Copy Markdown

Test262 conformance changes

Test result main count PR count difference
Total 52,963 52,963 0
Passed 50,126 50,126 0
Ignored 2,025 2,025 0
Failed 812 812 0
Panics 0 0 0
Conformance 94.64% 94.64% 0.00%

Tested main commit: be4cde434e03a9b3b117dc1f39ae394efffdfb54
Tested PR commit: fd81d41eb52664b1c34c167c50873aaa3bfd4b2a
Compare commits: be4cde4...fd81d41

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.80%. Comparing base (6ddc2b4) to head (fd81d41).
⚠️ Report is 903 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #5180       +/-   ##
===========================================
+ Coverage   47.24%   59.80%   +12.55%     
===========================================
  Files         476      582      +106     
  Lines       46892    63414    +16522     
===========================================
+ Hits        22154    37923    +15769     
- Misses      24738    25491      +753     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Member

@jedel1043 jedel1043 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, thank you for the fix

@jedel1043 jedel1043 added this pull request to the merge queue Mar 20, 2026
@jedel1043 jedel1043 added A-Technical Debt Changes related to technical debt A-Meta Issues and PRs related to the repository itself and removed A-Technical Debt Changes related to technical debt labels Mar 20, 2026
Merged via the queue into boa-dev:main with commit 94c9300 Mar 20, 2026
22 checks passed
@jedel1043 jedel1043 removed the C-Builtins PRs and Issues related to builtins/intrinsics label Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Meta Issues and PRs related to the repository itself C-Actions Pull requests that update Github Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: Replace bare cargo install with cached baptiste0928/cargo-install

2 participants