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

Hudson commented on AVRO-2833:
------------------------------

SUCCESS: Integrated in Jenkins build AvroJava #862 (See 
[https://builds.apache.org/job/AvroJava/862/])
AVRO-2833: Fix python whitespace for lint. (#881) (github: 
[https://github.com/apache/avro/commit/e813aff84591724b237c1943e5d20b27d8d50bf4])
* (edit) lang/py/avro/test/test_datafile_interop.py


> [Python] lint fails with pycodestyle 2.6.0
> ------------------------------------------
>
>                 Key: AVRO-2833
>                 URL: https://issues.apache.org/jira/browse/AVRO-2833
>             Project: Apache Avro
>          Issue Type: Bug
>            Reporter: Ryan Skraba
>            Assignee: Ryan Skraba
>            Priority: Major
>             Fix For: 1.10.0
>
>
> It looks like our code fails the lint since the recent update of pycodestyle 
> from 2.5.0 to 2.6.0.
> {code}
> +cd lang/py
> + ./build.sh lint test
> GLOB sdist-make: /home/travis/build/RyanSkraba/avro/lang/py/setup.py
> lint create: /home/travis/build/RyanSkraba/avro/lang/py/.tox/lint
> lint installdeps: isort, pycodestyle
> lint inst: 
> /home/travis/build/RyanSkraba/avro/lang/py/.tox/.tmp/package/1/avro-1.10.0+snapshot.zip
> lint installed: avro @ 
> file:///home/travis/build/RyanSkraba/avro/lang/py/.tox/.tmp/package/1/avro-1.10.0%2Bsnapshot.zip,isort==4.3.21,pycodestyle==2.6.0
> lint run-test-pre: PYTHONHASHSEED='3539139953'
> lint run-test: commands[0] | isort --check-only
> Skipped 1 files
> lint run-test: commands[1] | pycodestyle
> ./avro/test/test_datafile_interop.py:30:1: E302 expected 2 blank lines, found 
> 1
> 1       E302 expected 2 blank lines, found 1
> ERROR: InvocationError for command 
> /home/travis/build/RyanSkraba/avro/lang/py/.tox/lint/bin/pycodestyle (exited 
> with code 1)
> {code}
> This can be reproduced locally (and on all PR checks since May 11th).
> We could pin pycodestyle to 2.5.0, but we should also listen to the lint and 
> fix this whitespace.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to