pombreda / formalchemy

Automatically exported from code.google.com/p/formalchemy
MIT License
0 stars 0 forks source link

Bunch of patches #105

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Here are a bunch of patches I'm using in production at some place. Hope it
gets into the core.

You can follow those patches on my git repository, on github at
http://github.com/abourget/formalchemy-abourget/tree/master

Note to jbellis: being able to call  .configure() multiple times isn't
used. So it's not a big deal if you don't get it in.

I didn't find another place to post this. I know it's not a defect.

Original issue reported on code.google.com by wackysa...@gmail.com on 19 Jun 2009 at 6:24

Attachments:

GoogleCodeExporter commented 9 years ago
I have a quick look at some patches.

1/ this is very hard to follow...

2/ all api changes need to be discussed on the mailing list

3/ patches i've opened don't provide unit tests.

Result: can't apply

Don't take this message as a blame. You effort and interest is appreciate. 
There is
just too much changes without discuss/tests.

Regards,

Original comment by gael.pas...@gmail.com on 28 Jun 2009 at 10:03