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
Investigate unused imports in tests/* #916
Conversation
367f326
to
18acf26
Compare
This is a great start; thank you so much! To prevent these from creeping back in, I've re-enabled flake8 on the test folder on this branch. It'll probably find a bunch more issues. If you can fix them, we'd really appreciate it. If not, I'll be back to clean them up when I get the chance. |
You're welcome. I'd be happy to keep the changes coming. |
@SamWilsn are there any more implementations needed for this pr for it to be merged? |
Tests passed locally! |
What was wrong?
There are unused imports in the test folder. Refactoring needed
Related to Issue #
closes #901
How was it fixed?
Use pylint to find unused import and refactor it
Cute Animal Picture