-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Rust, ruby: Update cargo dependencies #20553
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates Cargo dependencies for Rust and Ruby extractors to their latest compatible versions. The update includes minor version bumps for various serialization, regular expression, and CLI-related crates.
Key Changes
- Updated core dependencies like
serdefrom 1.0.219 to 1.0.228,regexfrom 1.11.2 to 1.11.3, andclapfrom 4.5.47 to 4.5.48 - Updated tree-sitter-embedded-template from 0.23.2 to 0.25.0 for Ruby extractor
- Updated build system and Bazel configuration files to reflect the new dependency versions
Reviewed Changes
Copilot reviewed 89 out of 90 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| shared/tree-sitter-extractor/Cargo.toml | Updated regex dependency to 1.11.3 |
| rust/extractor/macros/Cargo.toml | Updated quote dependency to 1.0.41 |
| rust/extractor/Cargo.toml | Updated multiple dependencies including anyhow, clap, serde, serde_json, and toml |
| rust/ast-generator/Cargo.toml | Updated quote, serde, and anyhow dependencies |
| ruby/extractor/Cargo.toml | Updated tree-sitter-embedded-template, regex, and serde_json dependencies |
| misc/bazel/3rdparty/tree_sitter_extractors_deps/defs.bzl | Updated Bazel dependency mappings to match new versions |
| misc/bazel/3rdparty/tree_sitter_extractors_deps/BUILD.*.bazel | Updated numerous Bazel build files with new dependency versions and configurations |
| MODULE.bazel | Updated Bazel module references to new dependency versions |
geoffw0
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm still waiting for a fully working Rust DCA run on this PR. Otherwise happy with it.
|
Thanks for the ping @geoffw0. I had missed your other comment. I ran the retry command, and as far as I can tell the DCA report now looks fine. |
Updates cargo dependencies for Rust and Ruby.
I think I've generated the reports with the correct flags. The Ruby report complains about stage timings, with I can't really judge.