Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@dlstadther
Copy link
Collaborator

Description

Fixes flake8 error on master branch

Motivation and Context

Failing master build

Have you tested this? If so, how?

Ran flake8 tests locally.

Before change:

flake8 run-test: commands[0] | flake8 --exclude=doc,.tox
./test/worker_test.py:1362:118: E231 missing whitespace after ','
ERROR: InvocationError for command /Users/dlstadther/code/luigi/.tox/flake8/bin/flake8 --exclude=doc,.tox (exited with code 1)
ERROR:   flake8: commands failed

After change:

flake8 run-test: commands[1] | flake8 --max-line-length=100 --ignore=E265 doc
  flake8: commands succeeded
  congratulations :)

@dlstadther dlstadther requested a review from a team as a code owner August 18, 2023 00:34
@RRap0so RRap0so merged commit ee2fa4b into spotify:master Aug 20, 2023
@dlstadther
Copy link
Collaborator Author

Thanks for the merge, @rafaelraposospot !

@dlstadther dlstadther deleted the fix-master-lint branch August 20, 2023 19:45
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.

2 participants