amol- commented on pull request #7732: URL: https://github.com/apache/arrow/pull/7732#issuecomment-879868157
A good first step might be to ensure no new ones can be added. In other contexts I used to `wc -l` the output of flake8 _before_ and _after_ the PR (thus on master and on the PR) to ensure that the PR didn't introduce any _new_ error. A similar strategy might be applied here, while we fix the existing ones, we can put in place a check that prevents new ones from occurring by simply counting the number of existing errors (or if you want to go further you can compare the list of functions with errors, but In my experience just comparing the count does 90% of the job) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
