Code review comment for lp:~cprov/core-image-watcher/lint

Revision history for this message
Francis Ginther (fginther) wrote :

I think we should consider pylint as an option in the next sprint. As late as it is in the current sprint, I think it will end up being a distraction trying to get already existing code to comply and not produce it's intended benefits (which could be good benefits) and we don't have that much new code to write for this sprint.

But since this work is already done, I'm ok landing it and seeing how it works. If we want to invest more pylint in the future, we need to start somewhere to know if it's actually going to help.

review: Approve

« Back to merge proposal