Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix flake8 error in master #3248

Merged
merged 1 commit into from
Aug 20, 2023
Merged

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
35 checks passed
@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.

None yet

2 participants