Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix to not specify the rust toolchain version everywhere #5614

Merged
merged 5 commits into from
Mar 4, 2025

Conversation

jtran
Copy link
Collaborator

@jtran jtran commented Mar 3, 2025

Follow-up to #5606.

This also fixes target dir caching. We weren't specifying the root of our Rust workspace, so our target directory was never being cached.

Options were removed because after moving to actions-rust-lang/setup-rust-toolchain:

  • When no toolchain option is used, it looks for the rust-toolchain.toml file.
  • override: true is the default.
  • There is no default option.
  • Swatinem/rust-cache is built-in.

Copy link

qa-wolf bot commented Mar 3, 2025

QA Wolf here! As you write new code it's important that your test coverage is keeping up.
Click here to request test coverage for this PR!

Copy link

vercel bot commented Mar 3, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
modeling-app ✅ Ready (Inspect) Visit Preview Mar 4, 2025 2:50am

jtran added 2 commits March 3, 2025 18:50
@jtran jtran force-pushed the jtran/fix-rust-toolchain branch from 166153e to 33cf999 Compare March 4, 2025 00:21
Copy link

codecov bot commented Mar 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.31%. Comparing base (6e57a80) to head (71e49f0).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5614   +/-   ##
=======================================
  Coverage   86.31%   86.31%           
=======================================
  Files         102      102           
  Lines       40294    40294           
=======================================
  Hits        34778    34778           
  Misses       5516     5516           
Flag Coverage Δ
rust 86.31% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Copy link
Contributor

@lf94 lf94 left a comment

Choose a reason for hiding this comment

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

LGTM. Uses @iterion 's original suggestion

Copy link
Contributor

@maxammann maxammann left a comment

Choose a reason for hiding this comment

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

looks good except for the updated snapshot images. not sure about those

@lf94
Copy link
Contributor

lf94 commented Mar 4, 2025

@jtran force push

@jtran jtran force-pushed the jtran/fix-rust-toolchain branch from 71e49f0 to 54e94c5 Compare March 4, 2025 18:55
@jtran jtran enabled auto-merge (squash) March 4, 2025 18:57
@jtran jtran merged commit 725c56e into main Mar 4, 2025
66 of 77 checks passed
@jtran jtran deleted the jtran/fix-rust-toolchain branch March 4, 2025 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants