-
Notifications
You must be signed in to change notification settings - Fork 111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
First draft for Go quickstart guide #530
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Amazing!
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #530 +/- ##
===========================================
- Coverage 79.36% 45.22% -34.15%
===========================================
Files 69 67 -2
Lines 5772 5597 -175
===========================================
- Hits 4581 2531 -2050
- Misses 970 2914 +1944
+ Partials 221 152 -69
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
```sh | ||
export OTEL_EXPORTER_OTLP_ENDPOINT=https://otlp-gateway-prod-eu-west-0.grafana.net/otlp | ||
export GRAFANA_CLOUD_INSTANCE_ID=123456 | ||
export GRAFANA_CLOUD_API_KEY="your api key here..." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed: #538
@mariomac in the future would it be possible to complete a technical writing review before merging? |
@grafsean sure! Sorry for merging it too early |
@mariomac thank you 😄 not a problem! I'll share feedback in a new PR. In the meantime could we address Gregor's comment about API keys #530 (review) |
No description provided.