Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/7241#issuecomment-118937055
A few off-the-cuff thoughts (will have time to comment in more detail
later):
- We should exclude the heapq3 module for now.
- We need to redefine the regex used for matching names so that it doesn't
complain about Java-style camelCased variable and method names.
- We might want to disable the single-character variable name warning;
let's leave it up to reviewer judgement to handle those.
- We should disable "too many lines in module" warning.
Once we disable the noisy variable name regex warnings, we'll be able to
get a better sense of how many legitimate warnings we have.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
