Fork me on GitHub
#ring-swagger
<
2017-10-30
>
mrchance16:10:09

@ikitommi is there a special reasoning why :- is required for e.g. query-params, but leads to errors if used for the body? It's pretty confusing that the handling is different from place to place

ikitommi16:10:25

@mrchance just history. *-params use the plumbing/letk binding (good for asserting single params) while :body, :query and :headers use let binding (good for asserting the whole thing).

mrchance16:10:36

Ah, ok.. I thought it would be something along those lines. Not sure if the advantages outweigh the surprises, but probably hard to change now anyway 😉

ikitommi16:10:24

I think it could be changed, but Colin would need to update the static analyzing part in Cursive too.

ikitommi16:10:12

so, this would be better? :body [user :- User]

ikitommi16:10:22

If that works oob with Cursive, it would be easy to add (just strip the :- in the middle position)

mrchance16:10:07

Hm, that's probably the best way then, just make it more tolerant

ikitommi16:10:20

do you use Cursive?

mrchance16:10:35

Nevar 😉 emacs

ikitommi16:10:38

haha. I'll test that when at laptop

mrchance16:10:52

Cool, thanks

mrchance16:10:36

Insanely good user support as usual 🙂

mrchance16:10:59

The colleague who had the problem asks if he can buy you a beer somewhere

mrchance16:10:24

I'd be interested too, you already helped a lot in the past @ikitommi