search

hoodiehq / pouchdb-hoodie-api

:dog: Hoodie-like API for PouchDB
https://hoodiehq.github.io/pouchdb-hoodie-api
Apache License 2.0
44 stars 11 forks source link

missing curly braces #51

Closed janl closed 9 years ago

janl commented 9 years ago

https://github.com/hoodiehq/pouchdb-hoodie-api/blob/master/utils/to-object.js#L6 nasty!

gr2m commented 9 years ago

We only do this for return statments, where it's perfectly fine. It's much more read able.

gr2m commented 9 years ago

sorry didn't meant to close it :)

toh82 commented 9 years ago

Don't want to be to loud, since i'm a new here, but for me letting out the curlies looks like a bug. True, it's readable if you fammilar with it, but it's bit like fallthrough in switches, where you donno if it is an author mistake or wanted, hope you got me right.

gr2m commented 9 years ago

fixed via https://github.com/hoodiehq/pouchdb-hoodie-api/pull/52