-
Notifications
You must be signed in to change notification settings - Fork 13k
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
regression: type annotations needed #135670
Comments
If I'm reading that PR's thread correctly, it seems that crater had some issue that made it not properly test some crates, but then the PR got merged anyway for some reason. |
Yep, there was a period of time when we were getting a lot of "unknown" crater results. That was fixed sometime later, and we didn't realize that it caused a bunch of false negatives until after. I think this is probably worth still breaking, though perhaps lcnr can characterize in a more clear way why this broke and why it's consistent to break it. |
the crate was published after the crater run on dec. 11 |
Note: if we are going to accept this breakage, #132325 will need |
I've opened a fix in LuisDuarte1/diesel-d1#2. I consider this to very much be intended breakage. cc @rust-lang/types I intend to close this regression as accepted. |
The text was updated successfully, but these errors were encountered: