-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Update rustc_pattern_analysis #16738
Conversation
@@ -88,7 +88,7 @@ ra-ap-rustc_lexer = { version = "0.35.0", default-features = false } | |||
ra-ap-rustc_parse_format = { version = "0.35.0", default-features = false } | |||
ra-ap-rustc_index = { version = "0.35.0", default-features = false } | |||
ra-ap-rustc_abi = { version = "0.35.0", default-features = false } | |||
ra-ap-rustc_pattern_analysis = { version = "0.37.0", default-features = false } | |||
ra-ap-rustc_pattern_analysis = { version = "0.40.0", default-features = false } |
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.
ra-ap-rustc_pattern_analysis = { version = "0.40.0", default-features = false } | |
ra-ap-rustc_pattern_analysis = { version = "0.41.0", default-features = false } |
I don't think 0.40 actually includes your fix, and 0.41 will hopefully come out in a couple of hours.
We should probably bump that workflow to run earlier 😀.
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 checked, it does include the fix :) It was published shortly after the fix landed, to my own surprise
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.
Ah, sorry then. Still, can you bump it to the new version? It's out now.
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.
Just saw that ^^ Yeah I'll bump next time it's useful
Eh, whatever. We can upgrade next time. @bors r+ |
☀️ Test successful - checks-actions |
This should fix #16656 but I can't check because we don't have a reproducer.