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 #360

Closed ellbosch closed 4 years ago

ellbosch commented 4 years ago

Last annoying PR for 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 introduces 1 alert and fixes 1 when merging 3bcb0878bc71db8193373cb2d50c8181c2b30a88 into 9cb84e21ac50f2082c1c136780b674cb35a0c5d8 - view on LGTM.com

new alerts:

1 for Unmatchable dollar in regular expression

fixed alerts:

1 for Wrong number of arguments for format

lgtm-com[bot] commented 4 years ago

This pull request fixes 1 alert when merging 6c137873c21b807a3a78d4e14b78d7f60345c9ed into 9cb84e21ac50f2082c1c136780b674cb35a0c5d8 - view on LGTM.com

fixed alerts:

1 for Wrong number of arguments for format

ellbosch commented 4 years ago

Will explore test failures and send update.