-
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
lint on a call to exit
in tests
#966
Comments
@Manishearth We should have a T-Test-related label. |
Well, T- is for categorizing how the lint is to be written, and is basically used to make the E- labels more finer grained. But +1 for A-Test and A-style and whatnot. Or some other prefix (servo uses A-) |
Fine with me. |
Is this still a potential lint? I'd like to give it a try! :) |
While still a potential lint, Without cross crate analysis capabilities or MIR-only-rlibs, any implementation we could do is pretty restricted |
Hmm. I see. Are those things coming? If not, maybe this can be tagged or otherwise noted somewhere as "good idea but necessary tooling isn't coming up anytime soon"? |
cc rust-lang/rust#33954
Should be easy-enough once @llogiq's cross crate analyzer is up and running.
The text was updated successfully, but these errors were encountered: