Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

add transaction dependency to zodb scaffold #716

Merged
merged 2 commits into from Nov 28, 2012

Conversation

Projects
None yet
2 participants
Contributor

ericrasmussen commented Nov 6, 2012

Updating zodb scaffold with explicit dependency on the transaction package.

Owner

mcdonc commented Nov 6, 2012

Thanks. This simple change necessitates downstream changes to the wiki tutorial, so it may be a while before I get the gumption to apply it.

Contributor

ericrasmussen commented Nov 6, 2012

Sorry about that -- didn't mean to make more work for you. I updated it in all the docs and made sure all the tests still pass. This is a non-urgent pull request but if you need anything else let me know!

Owner

mcdonc commented Nov 14, 2012

Every time I go to merge this I bounce off it, because it means I have to make sure that the line numbers mentioned in the tutorial rst files match real line numbers. I almost got up the gumption a second ago, but it was fleeting and I decided to do something else. Anyway, eventually I'll merge it.

mcdonc added a commit that referenced this pull request Nov 28, 2012

Merge pull request #716 from ericrasmussen/scaffolds.requiresfix
add transaction dependency to zodb scaffold

@mcdonc mcdonc merged commit 090a384 into Pylons:master Nov 28, 2012

Owner

mcdonc commented Nov 28, 2012

Cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment