search

inveniosoftware / dojson

Simple pythonic JSON to JSON converter.
https://dojson.readthedocs.io
Other
10 stars 29 forks source link

utils: force_list roundtrip check #149

Closed jacquerie closed 8 years ago

jacquerie commented 8 years ago

A simple fix for some inconsistency I spotted.

CC: @kaplun

coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.02%) to 82.796% when pulling 7469f223bb76cbb07b6ce12a25c97dbeb51d8e29 on jacquerie:force-list-roundtrip into 117fa7421059a859a42e1b8edf7b4b750e297900 on inveniosoftware:master.

jacquerie commented 8 years ago

Failure on Travis appears unrelated to the PR.

lnielsen commented 8 years ago

@jacquerie Due to Sphinx 1.4 - see latest commit in cookiecutter-invenio-module for fix.

jacquerie commented 8 years ago

Currently on vacation - will look at this on Monday.

On Fri, Jul 8, 2016, 08:48 Jiri Kuncar notifications@github.com wrote:

Assigned #149 https://github.com/inveniosoftware/dojson/pull/149 to @jacquerie https://github.com/jacquerie.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/inveniosoftware/dojson/pull/149#event-716654721, or mute the thread https://github.com/notifications/unsubscribe/AAXRYLpH1ogxgjEyDoE9-JzVdJhc40H3ks5qTfK_gaJpZM4IqVbl .

jacquerie commented 8 years ago

Will wait for #155 to be merged, because it probably won't be necessary after that.