search

bitly / bitly-api-python

python library to the bitly api
http://github.com/bitly/bitly-api-python
Other
245 stars 92 forks source link

PEP8-ify bitly_api.py #23

Closed jsocol closed 11 years ago

jsocol commented 11 years ago

Run the flake8 tool and fix until it doesn't report any issues.

jsocol commented 11 years ago

@mccutchen Mind taking a quick look?

mccutchen commented 11 years ago

@jsocol LGTM… is it worth investigating a pre-commit hook (or some other way) to enforce this style?

jsocol commented 11 years ago

Since we can't add hooks to github it would have to be installed locally for everyone, though if we can get things onto travis-ci it should be possible to add style checks to the normal test suite

mccutchen commented 11 years ago

Yeah, cool, let's just keep it in mind if/when we do get this hooked up to something like travis.

jsocol commented 11 years ago

@mccutchen mind merging?