Skip to content

Conversation

thaJeztah
Copy link
Member

@thaJeztah
Copy link
Member Author

Argh... guess TEST_IMG instead of TEST_MAGE would work 🤷‍♂

flake8 run-test: commands[0] | flake8 docker tests setup.py
223tests/integration/regression_test.py:38:80: E501 line too long (80 > 79 characters)
224tests/integration/api_container_test.py:284:80: E501 line too long (81 > 79 characters)
225tests/integration/api_container_test.py:936:80: E501 line too long (80 > 79 characters)
226tests/integration/api_container_test.py:949:80: E501 line too long (80 > 79 characters)
227tests/integration/api_container_test.py:1363:80: E501 line too long (80 > 79 characters)
228tests/integration/api_image_test.py:80:80: E501 line too long (80 > 79 characters)
229tests/integration/api_image_test.py:115:80: E501 line too long (80 > 79 characters)
230ERROR: InvocationError for command /home/travis/build/docker/docker-py/.tox/flake8/bin/flake8 docker tests setup.py (exited with code 1)
231___________________________________ summary ____________________________________
232ERROR:   flake8: commands failed
233The command "tox" exited with 1.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Copy link
Contributor

@ulyssessouza ulyssessouza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shin- shin- merged commit 845d31d into docker:master Aug 12, 2019
@shin-
Copy link
Contributor

shin- commented Aug 12, 2019

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants