-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[ty] Update salsa #22072
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
[ty] Update salsa #22072
Conversation
Diagnostic diff on typing conformance testsNo changes detected when running ty on typing conformance tests ✅ |
|
|
Need to update cargo fuzz too. I'll do that once the upstream PR is merged |
|
006474c to
eca2da8
Compare
|
Hi, |
|
There are a few more salsa changes that we want to get in, where pinning is very convenient, but we do plan to move to releases eventually. |
Summary
Salsa now enforces that interned structs and many-argument queries implement
salsa::Update(or have a&'staticlifetime).This prevents unsoundness bugs, but requires us to derive
salsa::Updatein a few more places.Note, this requires salsa-rs/salsa#1041 or we have to manually implement
salsa::Updatefor the tuple types, which is a bit painful.Test Plan
cargo test