juxt / bidi

Bidirectional URI routing
MIT License
993 stars 90 forks source link

Update README.md #200

Closed cloojure closed 4 years ago

cloojure commented 4 years ago

Correct spacing on routes with guards.

SevereOverfl0w commented 4 years ago

Shouldn't /zip be less indented than that?

cloojure commented 4 years ago

so the zip map is aligned with the :get map

On Thu, Jan 30, 2020 at 2:17 AM Alan Thompson cloojure@gmail.com wrote:

Hi - I wasn't sure, so I pasted it into IntelliJ/Cursive, and that is the result..

Basically, the map with with :request-method is a key on the same level as the string "blog". So you get

{"blog"

} and it aligns both the 2 keys and the 2 values in the map. On Thu, Jan 30, 2020 at 1:26 AM Dominic Monroe wrote: > Shouldn't /zip be less indented than that? > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > , > or unsubscribe > > . >
cloojure commented 4 years ago

Hi - I wasn't sure, so I pasted it into IntelliJ/Cursive, and that is the result..

Basically, the map with with :request-method is a key on the same level as the string "blog". So you get

{"blog"

} and it aligns both the 2 keys and the 2 values in the map. On Thu, Jan 30, 2020 at 1:26 AM Dominic Monroe wrote: > Shouldn't /zip be less indented than that? > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > , > or unsubscribe > > . >
SevereOverfl0w commented 4 years ago

I've aligned it to what I think makes sense, I suspect cursive is trying to align the nested forms.