search

walmartlabs / lacinia

GraphQL implementation in pure Clojure
http://lacinia.readthedocs.io/en/latest/
Other
1.82k stars 160 forks source link

Include operation name in parsed query #430

Closed robhanlon22 closed 8 months ago

robhanlon22 commented 1 year ago

Useful for debug, diagnostic, or metrics purposes.

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

hlship commented 1 year ago

I can't look at this until the CLA is signed.

robhanlon22 commented 1 year ago

@hlship CLA signed! Sorry about the delay.

hlship commented 1 year ago

This seems incomplete; there's no tests, and no API for extracting the named operation from the map. I'm just not sure what's the point of this change.

hlship commented 8 months ago

Please see notes above and reopen with tests, and a rationale.