newrelic / opentelemetry-exporter-go

New Relic's Golang OpenTelemetry Exporter
Apache License 2.0
29 stars 28 forks source link

Proposed changes to the environment variables section #59

Closed rhetoric101 closed 3 years ago

rhetoric101 commented 3 years ago

I'd like to propose that we put some endpoint documentation in the code so it gets rendered in Godoc. The rationale for this is to help users with New Relic Infinite Tracing.

I see two similar inline documentation sections about environment variables in this code, so I wasn't sure which one (or both) to change. To get started, I chose the second section starting at line #134. Basically, I revised this section a bit to clarify the prerequisites, and I added a link to the "OpenTelemetry: Advanced configuration" guide at docs.newrelic.com, which has information about changing endpoints.

Could you confirm that this information is correct, in the correct format, and resides in the right place (for example, should it also be in the section starting in line #69)?