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.

Sign up for GitHub

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

Jump to bottom

Rust 2024 and 1.85 #19236

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Rust 2024 and 1.85 #19236

wants to merge 14 commits into from
+2,322 −2,571

Conversation

BenjaminBrienen
Copy link
Contributor

No description provided.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 26, 2025
@BenjaminBrienen
Copy link
Contributor Author

I hope this is more reviewable than the last one! I understand that this will not be merged until #18964 is landed. Please just block this until then. I will fix the merge conflicts afterward. :)

@BenjaminBrienen
Copy link
Contributor Author

This test fails for me, and I don't know why:

test tests::smoke_test_real_sysroot_cargo ... FAILED

failures:

---- tests::smoke_test_real_sysroot_cargo stdout ----

thread 'tests::smoke_test_real_sysroot_cargo' panicked at crates/base-db/src/input.rs:527:74:
no entry found for key

@ChayimFriedman2
Copy link
Contributor

It's not you. #19214.

@Veykril
Copy link
Member

Veykril commented Feb 27, 2025 •
edited
Loading

Gonna repeat myself just to be safe but this will have to land after #18964

Edit: Ah I see you raised that in a comment already, sorry :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChayimFriedman2 ChayimFriedman2 ChayimFriedman2 requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@BenjaminBrienen @ChayimFriedman2 @Veykril @rustbot