search

inveniosoftware-attic / jsonalchemy

JSONAlchemy
GNU General Public License v2.0
0 stars 10 forks source link

implement json encoder #31

Closed jacquerie closed 9 years ago

jacquerie commented 9 years ago

Following https://github.com/jirikuncar/jsonalchemy/pull/2#issuecomment-97719659, here is the PR.

jirikuncar commented 9 years ago

Shall we also implement general .dump() method that generates pure python representation?

json.dumps(JSONDict({}).dumps())  # or .to_python()
jirikuncar commented 9 years ago

@jacquerie can you polish the commit message? Can you fix the tests - it looks like trivial docs and PEP8 issues. Thanks

jacquerie commented 9 years ago

Whoops, forgot to run pep8 before committing. Also, maybe it's better if I use six.iteritems instead of dict.iteritems. Is the commit message better now?