Skip to content

skip analyzing *_test.go files or at least make an explicit mode that's opt-in #29

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
Closed
odeke-em opened this issue Nov 6, 2020 · 0 comments · Fixed by #30
Closed

skip analyzing *_test.go files or at least make an explicit mode that's opt-in #29

odeke-em opened this issue Nov 6, 2020 · 0 comments · Fixed by #30
Labels
enhancement New feature or request

Comments

@odeke-em
Copy link
Member

odeke-em commented Nov 6, 2020

*_test.go files will cause lots of noise as per golang/go#42415 (comment), thus perhaps let's add an explicit mode that the user has to enable, but by default let's skip over *_test.go files.
@odeke-em odeke-em added the enhancement New feature or request label Nov 6, 2020
cuonglm pushed a commit that referenced this issue Nov 6, 2020
@cuonglm
Ignore test files by default
36a397d 
To include test files in check, user can use flags -include-test-files.

Fixes #29
@cuonglm cuonglm mentioned this issue Nov 6, 2020
Merged
cuonglm pushed a commit that referenced this issue Nov 6, 2020
@cuonglm
Ignore test files by default
093c647 
To include test files in check, user can use flags -include-test-files.

Fixes #29
cuonglm pushed a commit that referenced this issue Nov 6, 2020
@cuonglm
Ignore test files by default
9b7901f 
To include test files in check, user can use flags -include-test-files.

Fixes #29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Ignore test files by default orijtech/structslop
1 participant
@odeke-em