PEP 8 changes - Githubissues

dbcli / mssql-cli

A command-line client for SQL Server with auto-completion and syntax highlighting

BSD 3-Clause "New" or "Revised" License

1.35k stars 191 forks source link

PEP 8 changes #367

Closed ellbosch closed 4 years ago

ellbosch commented 4 years ago

Re-opening this PR with tests fixed.

Last PR for the laundry list of PEP 8. Closes #351.

You may notice that mssqlcli/packages/sqlcompletion.py is a bit of a mess--I've handled most PEP 8 issues here but I raised a separate issue to properly refactor it.

lgtm-com[bot] commented 4 years ago

This pull request fixes 1 alert when merging 5509ce12315088e058049faef5af871dcfe50ead into b10616114a54db4655b08d0dfc824697ecde405e - view on LGTM.com

fixed alerts:

1 for Wrong number of arguments for format

ellbosch commented 4 years ago

I'll merge this tomorrow unless anyone has suggestions they'd like to raise.