Skip to content

make scaffolds pep8 compliant #384

Closed
wants to merge 1 commit into from

2 participants

@gawel
Pylons Project member
gawel commented Dec 26, 2011

all in the title

@mcdonc
Pylons Project member
mcdonc commented Jan 6, 2012

FWIW, the impact of merging this would be a lot less if the tutorials didn't have copies of rendered scaffolds. As it is, I'd have to merge this, then make all of these changes 11 times. That gives me a case of the dont-wannas.

@gawel
Pylons Project member
gawel commented Jan 6, 2012

Can this be automated ? If yes, I'll try

@mcdonc
Pylons Project member
mcdonc commented Jan 6, 2012

The problem is that the tutorial text mentions line numbers in places, and later stages of the tutorials build on earlier ones. I've tried to get around doing that, but not mentioning line numbers makes the tutorials hard to follow.

In any case, if we were to also change the tutorials to make sure the line numbers they mention were correct, this would be fine.

@mcdonc
Pylons Project member
mcdonc commented Nov 28, 2012

This patch has unfortunately lost relevance. I'm sure there are still "pep 8 violations" but I'm not working too hard to squash them, apologies.

@mcdonc mcdonc closed this Nov 28, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.