ipwright commented on issue #23567: [MINOR][BUILD] ensure call to translate_component has correct number of arguments URL: https://github.com/apache/spark/pull/23567#issuecomment-455102150 @srowen Thanks for merging, and very glad to hear you like LGTM. I'm happy to submit PRs that fix the simple alerts, if that will save you time (please advise). I helped develop LGTM's code quality algorithms, and so I'm interested in knowing how much fixing and clean-up effort is required to move a project up a grade. E.g., Spark is doing good: [](https://lgtm.com/projects/g/apache/spark/context:java) [](https://lgtm.com/projects/g/apache/spark/context:javascript) [](https://lgtm.com/projects/g/apache/spark/context:python) However, the top possible rating is A+. And so I want to get a handle on the effort required to get the top grade. I recommend enabling LGTM automated code review on Apache Spark, since it detects problems prior to merging. https://lgtm.com/help/lgtm/managing-pr-integration#enabling-pr-integration Best wishes, Ian.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
