search

akapur / pyiqfeed

Python LIbrary for reading DTN's IQFeed
GNU General Public License v2.0
168 stars 108 forks source link

Updated: All requested changes #10

Closed erickmiller closed 7 years ago

erickmiller commented 7 years ago

*Updated: Added more PEP8 fixes to conn.py, now when running
pylint --confidence=HIGH -E -s y conn.py

The pylint analysis results in:

Your code has been rated at 10.00/10

Also includes the previously requested fixes:

PEP compliance (ran static checking and fixing with pylint and autopep) Refactored DerivConn into BarConn Removed DerivativeInfo from library Reverted mkt_tm potentially obsolete bug fix

Also, sorry again for the manual merge, still not sure where the conflict is but it shouldn't be logical.

erickmiller commented 7 years ago

Closing this pull request as I've fixed this issue: https://github.com/akapur/pyiqfeed/issues/11
This pull request is obsolete and superseded by the following pull request: https://github.com/akapur/pyiqfeed/pull/12