search

aequitas / concourse-ftp-resource

Concourse resource to interact with FTP servers.
https://hub.docker.com/r/aequitas/ftp-resource/
MIT License
3 stars 6 forks source link

docker build is failing - E501 line too long #11

Closed drnic closed 5 years ago

drnic commented 5 years ago 
$ docker build -t drnic/concourse-ftp-resource .
...
Step 7/13 : RUN pylama /opt/resource /opt/resource-tests/
 ---> Running in 8fa3ff40f5d3
opt/resource/ftp.py:48:80: E501 line too long (87 > 79 characters) [pep8]
opt/resource/ftp.py:54:80: E501 line too long (97 > 79 characters) [pep8]
opt/resource/ftp.py:118:80: E501 line too long (103 > 79 characters) [pep8]
opt/resource/ftp.py:124:80: E501 line too long (84 > 79 characters) [pep8]
opt/resource/ftp.py:129:80: E501 line too long (85 > 79 characters) [pep8]
opt/resource/ftp.py:141:80: E501 line too long (83 > 79 characters) [pep8]
opt/resource/ftp.py:163:80: E501 line too long (101 > 79 characters) [pep8]
opt/resource/ftp.py:186:80: E501 line too long (86 > 79 characters) [pep8]
opt/resource/ftp.py:207:80: E501 line too long (83 > 79 characters) [pep8]
opt/resource/ftp.py:217:80: E501 line too long (83 > 79 characters) [pep8]
opt/resource/ftp.py:218:80: E501 line too long (88 > 79 characters) [pep8]
opt/resource/ftp.py:223:80: E501 line too long (86 > 79 characters) [pep8]
opt/resource/ftp.py:227:80: E501 line too long (85 > 79 characters) [pep8]
opt/resource/ftp.py:236:80: E501 line too long (96 > 79 characters) [pep8]
opt/resource/ftp.py:242:80: E501 line too long (84 > 79 characters) [pep8]
opt/resource-tests/test_out.py:16:80: E501 line too long (80 > 79 characters) [pep8]
opt/resource-tests/test_out.py:41:80: E501 line too long (80 > 79 characters) [pep8]
opt/resource-tests/test_out.py:55:80: E501 line too long (94 > 79 characters) [pep8]
opt/resource-tests/test_out.py:70:80: E501 line too long (80 > 79 characters) [pep8]
opt/resource-tests/test_out.py:71:80: E501 line too long (80 > 79 characters) [pep8]
opt/resource-tests/test_out.py:72:80: E501 line too long (80 > 79 characters) [pep8]
opt/resource-tests/test_out.py:103:80: E501 line too long (80 > 79 characters) [pep8]
opt/resource-tests/test_out.py:104:80: E501 line too long (80 > 79 characters) [pep8]
opt/resource-tests/test_out.py:148:80: E501 line too long (81 > 79 characters) [pep8]
opt/resource-tests/test_out.py:149:80: E501 line too long (81 > 79 characters) [pep8]
opt/resource-tests/test_out.py:150:80: E501 line too long (81 > 79 characters) [pep8]
opt/resource-tests/conftest.py:47:80: E501 line too long (81 > 79 characters) [pep8]
opt/resource-tests/test_misc.py:36:80: E501 line too long (94 > 79 characters) [pep8]
opt/resource-tests/test_misc.py:40:80: E501 line too long (82 > 79 characters) [pep8]
opt/resource-tests/test_check.py:46:80: E501 line too long (85 > 79 characters) [pep8]
opt/resource-tests/test_check.py:53:80: E501 line too long (94 > 79 characters) [pep8]
opt/resource-tests/test_check.py:65:80: E501 line too long (80 > 79 characters) [pep8]
opt/resource-tests/test_check.py:66:80: E501 line too long (86 > 79 characters) [pep8]
opt/resource-tests/test_check.py:70:80: E501 line too long (87 > 79 characters) [pep8]
opt/resource-tests/test_check.py:81:80: E501 line too long (80 > 79 characters) [pep8]
opt/resource-tests/test_check.py:84:80: E501 line too long (91 > 79 characters) [pep8]
opt/resource-tests/test_check.py:95:80: E501 line too long (86 > 79 characters) [pep8]
opt/resource-tests/test_check.py:96:80: E501 line too long (88 > 79 characters) [pep8]
opt/resource-tests/test_check.py:101:80: E501 line too long (80 > 79 characters) [pep8]
opt/resource-tests/test_check.py:116:80: E501 line too long (88 > 79 characters) [pep8]
opt/resource-tests/test_check.py:117:80: E501 line too long (86 > 79 characters) [pep8]
The command '/bin/sh -c pylama /opt/resource /opt/resource-tests/' returned a non-zero code: 1

Perhaps this E501 error is new to python in last year or so. Ideas on how to disable this check so i can build the docker image and investigate a PR for #10 ?

aequitas commented 5 years ago

https://github.com/aequitas/concourse-ftp-resource/commit/435faa8f101d273d6d360ec568b8c83abac325a8 should fix the issue with the linter. The syntax for the configuration of the line length was changed in some version I guess.