PEP 8 changes in JSON RPC

dbcli / mssql-cli

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

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

1.36k stars 191 forks source link

Closed ellbosch closed 4 years ago

ellbosch commented 4 years ago

Tag: #351

More PEP 8 changes, this time focusing on JSON RPC module. Roughly 65 pylint warnings are fixed in this PR, with an estimated 180 which remain.

ellbosch commented 4 years ago

Sorry everyone--I should have completed more tests before adding folks to review. I'll re-assess and send an update here.

ellbosch commented 4 years ago

The test failure is due to a flaky test. Otherwise, this PR is ready for review.