search

GoodCloud / django-zebra

Forms, widgets, template tags and examples that make Stripe + Django easier.
MIT License
194 stars 68 forks source link

Death to copy & paste. #12

Closed danawoodman closed 12 years ago

danawoodman commented 12 years ago

Fixing docstring to account for mislabeled properties.

leetrout commented 12 years ago

Heh, guilty as charged. But you need to update the lines following your changes in order to be accurate.

Would you care to do that and add the changes to this PR?

danawoodman commented 12 years ago

Think that should be it, no?

Don't worry, we're all guilty of such crimes once and a while ;)

leetrout commented 12 years ago

There's actually a lot wrong with all of those doc strings.... I just realized all of them say they add the property stripe but only the top one does.

Face Palm

danawoodman commented 12 years ago

We'll at least now you know of all your faults and you can address them :P

I'm still oblivious to most of mine...

leetrout commented 12 years ago

Haha, yea. Would you care to checkout PR 13 https://github.com/GoodCloud/django-zebra/pull/13 and let me know if something else jumps out at you?

I grabbed your changes with git pull git://github.com/danawoodman/django-zebra.git patch-1