Skip to content

Rust: upgrade rust-analyzer to 0.0.273 #19233

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

Merged
merged 9 commits into from
Apr 15, 2025
Merged

Rust: upgrade rust-analyzer to 0.0.273 #19233

merged 9 commits into from
Apr 15, 2025

Conversation

redsun82
Copy link
Contributor

@redsun82 redsun82 commented Apr 7, 2025

The new is_unsafe predicate added to struct fields is an experimental addition, more details in https://rust-lang.github.io/rust-project-goals/2025h1/unsafe-fields.html.

Commit by commit review is encouraged.

This includes #19291 which should therefore be merged before this.

@github-actions github-actions bot added the Rust Pull requests that update Rust code label Apr 7, 2025
@@ -14,4 +14,4 @@ members = [
[patch.crates-io]
# patch for build script bug preventing bazel build
# see https://github.com/rust-lang/rustc_apfloat/pull/17
rustc_apfloat = { git = "https://github.com/redsun82/rustc_apfloat.git", rev = "096d585100636bc2e9f09d7eefec38c5b334d47b" }
rustc_apfloat = { git = "https://github.com/redsun82/rustc_apfloat.git", rev = "32968f16ef1b082243f9bf43a3fbd65c381b3e27" }
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like your PR was merged 4 days ago. rust-lang/rustc_apfloat#17 I suppose we still need this patch for a little while, but hopefully we can drop it once a new version of rustc_apfloat is published

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, I don't know how long it will be till my patch gets released

@redsun82 redsun82 marked this pull request as ready for review April 14, 2025 14:35
@redsun82 redsun82 requested a review from a team as a code owner April 14, 2025 14:35
@redsun82 redsun82 requested a review from aibaars April 15, 2025 06:57
@redsun82
Copy link
Contributor Author

DCA shows some minor changes, but all seems acceptable to me.

@redsun82 redsun82 merged commit ff8cd98 into main Apr 15, 2025
35 checks passed
@redsun82 redsun82 deleted the redsun82/cargo-upgrade branch April 15, 2025 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants