-
Notifications
You must be signed in to change notification settings - Fork 93
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
allow textAnalyze to run on all files in a dir or matching a prefix #174
Conversation
doing so sort of breaks the 'invalid analyzer' message if no files match
…zing-all-files-in-dir
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 issues found.
getFiles := func(n string) (map[string][]byte, error) { | ||
matching := make(map[string][]byte) | ||
for k, v := range test.files { | ||
if strings.HasPrefix(k, n) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using continue here is less conditionally indented and more readable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this one is relevant
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@CodeLingoBot reply @laverya nice catch! Next time you can just reply to the bot like this and it'll pop up on the rule chat. What minimum number of lines do you think we need within the if statement before wanting to avoid nesting? Also @marccampbell? Any thoughts on this (reply on the rule chat and I'll tighten up the rule linter accordingly).
fixes #173