Skip to content

Commit

Permalink
Update GraphQL README to note that called setTracerProvider is opti…
Browse files Browse the repository at this point in the history
…onal (#293)

This caused me some confusion because they node-sdk doesn't give me a tracer provider to pass in here.  But it turned out I don't have to call this at all.

Co-authored-by: Bartlomiej Obecny <bobecny@gmail.com>
  • Loading branch information
dobesv and obecny authored Dec 28, 2020
1 parent 5dddf1d commit 7a0c39f
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ const graphQLInstrumentation = new GraphQLInstrumentation({
// mergeItems: true,
});

graphQLInstrumentation.setTracerProvider(provider);
graphQLInstrumentation.setTracerProvider(provider); // optional; uses global tracer by default

graphQLInstrumentation.enable();

Expand Down

0 comments on commit 7a0c39f

Please sign in to comment.