search

graphql-elixir / graphql

GraphQL Elixir
Other
859 stars 47 forks source link

Validate that mandatory arguments are present #80

Closed freshtonic closed 8 years ago

freshtonic commented 8 years ago

@joshprice @markolson @aweiker This is ready for review.

freshtonic commented 8 years ago

All tests are passing now. All that's remaining is directive arg validation and some minor refactoring to land this validator.

joshprice commented 8 years ago

LGTM

freshtonic commented 8 years ago

@joshprice thanks :dancers: