[ 
https://issues.apache.org/jira/browse/ARROW-16531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17536112#comment-17536112
 ] 

Raúl Cumplido commented on ARROW-16531:
---------------------------------------

I have created a PR for this ticket to update the pre-commit hook and remove 
the cython linting config from flake8, as it is not supposed to work. I have 
also created a new ticket to find some way of linting cython separately: 
https://issues.apache.org/jira/browse/ARROW-16544

Let me know if that sounds ok.

> [Python] Lint rules do not seem to be getting enforced
> ------------------------------------------------------
>
>                 Key: ARROW-16531
>                 URL: https://issues.apache.org/jira/browse/ARROW-16531
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: Developer Tools, Python
>            Reporter: Weston Pace
>            Assignee: Raúl Cumplido
>            Priority: Major
>              Labels: pull-request-available
>         Attachments: python-lint-errs
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> It seems there are some legitimate linting errors in master.
> {noformat}
> (arrow-dev) pace@pace-desktop:~/dev/arrow$ archery lint --python
> INFO:archery:Running Python formatter (autopep8)
> INFO:archery:Running Python linter (flake8)
> /home/pace/dev/arrow/python/pyarrow/_parquet.pyx:156:80: E501 line too long 
> (80 > 79 characters)
> /home/pace/dev/arrow/python/pyarrow/_parquet.pyx:170:80: E501 line too long 
> (80 > 79 characters)
> /home/pace/dev/arrow/python/pyarrow/_parquet.pyx:242:77: W291 trailing 
> whitespace
> /home/pace/dev/arrow/python/pyarrow/_parquet.pyx:447:80: E501 line too long 
> (80 > 79 characters)
> /home/pace/dev/arrow/python/pyarrow/_parquet.pyx:447:81: W291 trailing 
> whitespace
> ...
> {noformat}



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to