Add instrumentation for github.com/graph-gophers/graphql-go about opentelemetry-go-contrib HOT 5 CLOSED

@MrAlias may I pick this up ?

from opentelemetry-go-contrib.

@ravilushqa @jay-manday @jjmengze @gtourkas
Could you please try to collaborate and select one of your PRs as a preferred one (and make other PRs as a draft) to make the reviewers' life easier? 😉

from opentelemetry-go-contrib.

@pellared
This issue how i see opened for graph-gophers/graphql-go lib, whereas mine and @jjmengze is for 99designs/gqlgen. I'm not sure that both lib should go in a one MR, and imo that would be harder for reviewers.

In situation with @jjmengze, our pr's are both for a one lib. I'm not experienced contributor, so i'm not pretty sure how to deal with that. I'm pretty long waiting for it, and I will be happy if we will collaborate around it.

My suggest is to contribute separately this one, and mine #761 :)

from opentelemetry-go-contrib.

@pellared

@ravilushqa sufficiently described the case: this PR for for graph-gophers/graphql-go, which is also what the issue #85 is about, and the other two for 99designs/gqlgen . Apart from the registry item, which I guess is the last step, the PR is imo ready for review.

from opentelemetry-go-contrib.

In accordance with the new instrumentation policy hosting guidelines I'm closing this.

from opentelemetry-go-contrib.