Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Lint test functions which aren't marked as cfg(test) #1719

Closed
clarfonthey opened this issue May 2, 2017 · 1 comment
Closed

Lint test functions which aren't marked as cfg(test) #1719

clarfonthey opened this issue May 2, 2017 · 1 comment
Labels
A-lint Area: New lints

Comments

@clarfonthey
Copy link

We could also potentially lint test functions which aren't in a module that's marked cfg(test).
@phansch phansch added the A-lint Area: New lints label Dec 11, 2018
@edmorley edmorley mentioned this issue Sep 1, 2021
Merged
@Alexendoo
Copy link
Member

A restriction lint for tests outside test modules was added

#[test] implies #[cfg(test)] so a lint wouldn't be needed for this particular case

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-lint Area: New lints
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Alexendoo @phansch @clarfonthey