Weston Schmidt

That's good feedback. Looking at the patch again, it is making a strdup() call for each integer even if it perfectly converts. I'm going to do some work on this...

It's a bit more code than I had hoped, but this provides 4 new functions and keeps the existing behavior otherwise. ```c cJSON_ParseWithLengthNumStringOpts(); cJSON_GetNumberValueAsString(); cJSON_CreateNumberAsString(); cJSON_SetNumberValueAsString(); ``` These will validate...

@VelGer or @Alanscut any thoughts?

@VelGer I don't have any strong opinions on the `cJSON_ParseOpts()` ... but I'd like to get the functionality generally in. I've been busy with some other things, but if you...

You can use this trick for comments (works with cJSON): ```json { "": "This is a comment.", "thing": { "": "This is another comment.", "": "You can even have multiple...

`abstractlogger` doesn't have a license, so that needs to be addressed for most folks to be able to consume `graphql-go-tools` today. https://github.com/jensneuse/abstractlogger/issues/1

Thanks, that makes sense. Would you accept a PR adding the documentation / compile time check for this? ```golang // Ensure the BatchFactory implements the DataSourceBatchFactory. var _ resolve.DataSourceBatchFactory =...

It's been a while, have any of the opinions changed? I need to be able to rotate keys & would rather use this code vs. write my own duplicate if...

@oxisto / @mfridman if you're not happy with this approach for supporting multiple keys, do you have a path that you'd like for this use case? The interface for providing...

I have a need for this for supporting yocto build communities. I'm exploring what it would take to write this.