Skip to content

Conversation

jameslamb
Copy link
Contributor

@jameslamb jameslamb commented May 11, 2020

Thanks for this great project!

I wanted to take a few minutes and make a small contribution, so I ran flake8 over this codebase. A lot of the warnings generated are things that should only be done optionally by maintainers (like formatting and whitespace), but I think I found a few non-controversial changes that would be helpful.

In this PR, I'd like to propose removing a few unused imports and one local variable that is created but never used in ScramClient.

Thanks for your time and consideration.


This change is Reviewable

@jameslamb
Copy link
Contributor Author

I've rebased this to master to pick up changes since it was first opened.

@jeffwidman jeffwidman merged commit 316da74 into dpkp:master Jul 11, 2020
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.

3 participants