Andy Nicholson

Results 17 comments of Andy Nicholson

Hit this today. @DarkArc 's hypothesis was correct for me, too :)

@jordwalke I had no trouble using `https` as the protocol: ``` yarn global add https://github.com/reasonml/reason-cli.git yarn global v0.20.3 warning No license field [1/4] 🔍 Resolving packages... [2/4] 🚚 Fetching packages......

+1, got bitten by this issue today. Very annoying when the syntax error occured nearly 1500 lines into the file!

I guess it's winding back to the start of the structure it's trying to parse, rather than flagging the error then and there...

Would be a nice addition, +1

Hi @michaelherold, I'm a little confused. Do you mean that an `OmniAuth::AuthHash` passed to your mutation ends up as a `HashWithIndifferentAccess` by the time it arrives?

Postel's Law, perhaps? Agree both that it's annoying and that it's a little fiddly to fix. Hacky I know, but would recreating the `AuthHash` inside your mutation work? Looks like...

Another cheeky solution that just occurred to me. Utterly untested: ``` class OauthCommand < Mutations::Command def self.input_filters OauthFilter.new end end class OauthFilter def filter(data) return [data, :hash] unless data.is_a?(OmniAuth::AuthHash) end...

No idea, I'm just a regular punter :) @cypriss ?