3 ms·
Automatically frustrate and annoy your contributors! Make people never want to work with you! PS. Don't do this, ensuring test success is one thing. Requiring
by hahainternet 13y ago
Automatically frustrate and annoy your contributors! Make people never want to work with you!
PS. Don't do this, ensuring test success is one thing. Requiring PEP8 validation is dumb.
- StavrosK 13y agoBecause the codebase shouldn't have one consistent style?
- chromejs10 13y agoA contributor is supposed to be following the coding guidelines of the project from the start. If they don't bother to do that, they absolutely should be rejected. Why is it hard to follow a consistent style?
- hahainternet 13y agoBecause just remembering all of PEP8's requirements (which are mostly unjustified) is a frustrating burden. Keeping a consistent style is one thing. Mandating adherence to PEP8 is way over the top.
- jlgreco 13y agoIdeally anything that you reject for should also be conveniently baked into the build system so that you can do things like `make validate` before pushing to save yourself the embarrassment.