raulcd commented on PR #14662: URL: https://github.com/apache/arrow/pull/14662#issuecomment-1410256388
The workflow that executes linting on PRs is not using pre-commit but `archery lint`: https://github.com/apache/arrow/blob/master/.github/workflows/dev.yml#L56 https://github.com/apache/arrow/blob/master/docker-compose.yml#L1670 We probably want to add (or follow up) a PR adding a new archery linter for cython to ensure this gets executed on the GitHub workflow on PRs: https://github.com/apache/arrow/blob/master/dev/archery/archery/cli.py#L252-L265 I can see `cython-lint` being used on pandas. I think this would be useful. @jorisvandenbossche what are your thoughts on adding this? -- 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]
