krcrouse commented on PR #13126: URL: https://github.com/apache/arrow/pull/13126#issuecomment-1228457138
Hi @jorisvandenbossche and @pitrou, I made a number of additions so that the autogenerated _compute.py code conformed with flake8, since it doesn't distinguish between hand written and autogenerated code. I also updated the generated function signatures to be compliant with Python 3.7. Please let me know how you would like to move forward on this / if you do. If I'm reading the workflow output correctly, I think the only issue right now is that `archery` is failing due to the lack of licenses in the ReSTructured text files, and I am not sure how that can be added as I don't know of any standard for adding licenses to ReST (output from workflow failure below) ```bash INFO:archery:Running Docker linter apache-rat license violation: python/docs/additions/compute/all.rst apache-rat license violation: python/docs/additions/compute/any.rst apache-rat license violation: python/docs/additions/compute/count.rst apache-rat license violation: python/docs/additions/compute/count_distinct.rst apache-rat license violation: python/docs/additions/compute/filter.rst apache-rat license violation: python/docs/additions/compute/index.rst apache-rat license violation: python/docs/additions/compute/indices_nonzero.rst apache-rat license violation: python/docs/additions/compute/mode.rst apache-rat license violation: python/docs/additions/test_example.rst ``` -- 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]
