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

Upgrade cargo dependencies #18789

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Upgrade cargo dependencies #18789

wants to merge 11 commits into from

Conversation

redsun82
Copy link
Contributor

@redsun82 redsun82 commented Feb 14, 2025

  • rust-analyzer also requires a newer rustc, and switching to edition 2024. We could have different editions per crate, but I preferred to upgrade all of them.
  • Edition 2024 requires rust 1.85.

@Copilot Copilot bot review requested due to automatic review settings February 14, 2025 14:33
@redsun82 redsun82 requested a review from a team as a code owner February 14, 2025 14:33

Choose a reason for hiding this comment

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

Copilot reviewed 97 out of 97 changed files in this pull request and generated no comments.

Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab. Learn more

@github-actions github-actions bot added the Rust Pull requests that update Rust code label Feb 14, 2025
@redsun82 redsun82 added the depends on internal PR This PR should only be merged in sync with an internal Semmle PR label Feb 14, 2025
@redsun82 redsun82 requested review from a team as code owners February 14, 2025 15:34
@redsun82 redsun82 marked this pull request as draft February 14, 2025 15:39
@redsun82 redsun82 marked this pull request as ready for review February 14, 2025 15:48
@redsun82 redsun82 removed the depends on internal PR This PR should only be merged in sync with an internal Semmle PR label Feb 18, 2025
@redsun82 redsun82 mentioned this pull request Feb 18, 2025
@redsun82 redsun82 force-pushed the redsun82/cargo-upgrade branch from fd0aadf to 7e0f05f Compare February 24, 2025 16:46
@redsun82 redsun82 added the depends on internal PR This PR should only be merged in sync with an internal Semmle PR label Feb 24, 2025
@redsun82 redsun82 requested a review from a team as a code owner February 24, 2025 16:56
esbena
esbena previously approved these changes Feb 25, 2025
Comment on lines +38 to +40
RUST_EDITION = "2024"

RUST_VERSION = "1.82.0"
RUST_VERSION = "1.85.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm assuming this is the only human edit in the PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not really, as there were a couple of compilation errors to fix, but I understand the confusion, so I've chopped up the PR in individual more digestible commits, which should make it clear now what was done.

tausbn
tausbn previously approved these changes Feb 25, 2025
Copy link
Contributor

@tausbn tausbn left a comment

Choose a reason for hiding this comment

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

Python bits look fine to me. 👍

@redsun82 redsun82 dismissed stale reviews from tausbn and esbena via bfcf9ea February 25, 2025 12:37
@redsun82 redsun82 force-pushed the redsun82/cargo-upgrade branch from 099311a to bfcf9ea Compare February 25, 2025 12:37
@redsun82 redsun82 force-pushed the redsun82/cargo-upgrade branch 2 times, most recently from a1a5cad to d9ecb62 Compare February 25, 2025 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
depends on internal PR This PR should only be merged in sync with an internal Semmle PR Python Ruby Rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants