chore: upgrade toolchain and deps#447
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (6)
🚧 Files skipped from review as they are similar to previous changes (3)
📝 WalkthroughWalkthroughUpdates project tooling and dependency versions: Rust toolchain to 1.95.0, Cargo dependency bumps (frc42_dispatch, fvm_shared), JavaScript tooling bumps (node/pnpm engines, pnpm, wrangler, tailwindcss), pnpm workspace allowBuilds added, and crate attribute ChangesDependency & Tooling Updates
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
ea36b80 to
de76215
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #447 +/- ##
=======================================
Coverage 39.31% 39.32%
=======================================
Files 42 42
Lines 2935 2919 -16
=======================================
- Hits 1154 1148 -6
+ Misses 1781 1771 -10 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
9abd3ef to
3f5e81e
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@mise.toml`:
- Around line 3-4: mise.toml declares node = "22" and pnpm = "11" but
package.json still allows lower versions; update the package.json "engines"
field so that "node" is ">=22" and "pnpm" is ">=11" (i.e., change engines.node
and engines.pnpm to match mise.toml) to prevent mismatched runtime/tooling;
ensure any engine-related checks (e.g., engineStrict or CI scripts referencing
node/pnpm versions) remain consistent with these updated values.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 10c7a1a6-a33c-4e9e-8576-7d7ac795cb45
⛔ Files ignored due to path filters (2)
Cargo.lockis excluded by!**/*.lockpnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (6)
Cargo.tomlmise.tomlpackage.jsonpnpm-workspace.yamlrust-toolchain.tomlsrc/lib.rs
3f5e81e to
f1bb4ae
Compare
Summary of changes
Changes introduced in this pull request:
Reference issue to close (if applicable)
Closes
Other information and links
Change checklist
adheres to the team's
documentation standards,
(if possible),
should be reflected in this document.
Summary by CodeRabbit