-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
[DO NOT MERGE] see what breaks when we remove this recursive-enum-discriminants hack #61723
Conversation
r? @estebank (rust_highfive has picked a reviewer for you, use r? to override) |
@bors try |
[DO NOT MERGE] see what breaks when we remove this recursive-enum-discriminants check
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
☀️ Try build successful - checks-travis |
@rust-lang/infra could we get a check-only crater run for this? |
@craterbot run mode=check-only |
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
There's no point in cratering this. Certain old winapi versions that are in use will fail, but they won't be tested on crater. see #49765 and linked issues/PRs |
Seriously, winapi? 😢 Do you think there is any point in issuing a deprecation warning such that, maybe in a year or two we can remove this accidental feature? (I am still assuming this is an accident -- the feature is incredibly niche, can be easily worked around by factoring the common code into a |
@craterbot abort |
🗑️ Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
No description provided.