-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Remove unused imports #552
Conversation
@mbeacom please remove those whitespace changes (sorry, same thing happened with my editor) so the PR only contains your code changes. We will deal with the whitespace removal in separate PR's. I can re-review once you update. thanks! |
cd32039
to
a322622
Compare
@mbeacom thanks! I just scanned your branch and it looks like there are still a few unused imports left. Can you remove those also in this PR? They occur in these 4 files:
flake8 scan results:
|
@cgoldberg Thanks for your time! I have made the requested changes to the test and example files I missed in my update. I didn't remove the imports from the Would you like me to refactor to remove the need for these init imports? |
I'm going to merge this as-is... any refactoring can be done in another PR. |
Thanks for the contribution! |
* Remove unused imports and assigned exceptions * Remove additional unused imports locustio#552
This PR removes unused imports and exception assignments throughout the project.
Removed whitespace trimming per @cgoldberg's request.