Skip to content
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

Update metrics.md #135

Merged
merged 1 commit into from
Apr 23, 2021
Merged

Conversation

natashanwright
Copy link

correcting broken documentation link

Requirements

  • I have added test coverage for new or changed functionality
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

Provide links to any issues in this repository or elsewhere relating to this pull request.

Describe the solution you've provided

Provide a clear and concise description of what you expect to happen.

Describe alternatives you've considered

Provide a clear and concise description of any alternative solutions or features you've considered.

Additional context

Add any other context about the pull request here.

correcting broken documentation link
@@ -2,7 +2,7 @@

[(Back to README)](../README.md)

You can configure the Relay Proxy to export statistics and route traces to Datadog, Stackdriver, and Prometheus. To learn about the available settings for each of these options, read [Configuration](./docs/configuration.md).
You can configure the Relay Proxy to export statistics and route traces to Datadog, Stackdriver, and Prometheus. To learn about the available settings for each of these options, read [Configuration](../configuration.md).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for noticing the broken link, but your replacement link is also broken: configuration.md is in the same directory as metrics.md, so this should be a single period and not a double one.

@eli-darkly
Copy link
Contributor

We're getting ready to do a Relay Proxy bugfix release, so I'm going to go ahead and merge this PR into a branch so that I can fix the remaining problem that I commented on and get the change into the release.

@eli-darkly eli-darkly changed the base branch from v6 to contrib April 23, 2021 19:56
Copy link
Contributor

@eli-darkly eli-darkly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still needs one change, but we'll take care of that after merging. Thanks again for bringing this to our attention.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants