Code review comment for lp:~adiroiban/pocket-lint/1237862-pep-257

Revision history for this message
Adi Roiban (adiroiban) wrote :

Thanks for the review.

pep257 is still young and unstable so I don't think that it needs to be an Ubuntu package.

I will push a new change removing the install requirement and also skipping the tests when pep257 is not there.

I will also revert the pdb_call variable, but I think that it needs a comment. Otherwise is just triggers a WTF moment.

« Back to merge proposal