Skip to content
This repository has been archived by the owner on Aug 4, 2023. It is now read-only.

Fix Flake8 warnings about code quality #102

Closed
wants to merge 3 commits into from
Closed

Fix Flake8 warnings about code quality #102

wants to merge 3 commits into from

Conversation

obulat
Copy link
Contributor

@obulat obulat commented Jun 9, 2021

Fixes WordPress/openverse#1731

This PR fixes the following problems with code quality in provider API scripts:

  • names shadowing built-ins (license)
  • variable that are used before they are declared
  • unused variables
  • unnecessary parentheses around tuples

This PR is a half of #67, split to make review easier.

Signed-off-by: Olga Bulat <obulat@gmail.com>
Signed-off-by: Olga Bulat <obulat@gmail.com>
Signed-off-by: Olga Bulat <obulat@gmail.com>
@obulat obulat requested a review from a team as a code owner June 9, 2021 09:51
@obulat obulat requested review from zackkrida and dhruvkb June 9, 2021 09:51
@obulat obulat mentioned this pull request Jun 9, 2021
@obulat
Copy link
Contributor Author

obulat commented Jun 9, 2021

Replaced with #103 because this PR had unrelated changes.

@obulat obulat closed this Jun 9, 2021
@obulat obulat deleted the flake8 branch June 10, 2021 07:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Fix flake8 warnings
1 participant