-
Notifications
You must be signed in to change notification settings - Fork 222
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 dotnet-autoinstrumentation role link #883
Conversation
Before this change the link pointed to the top level readme which did not contain the community roles as indicated. Now the link points to the correct location where information can be found about membership and roles.
community-members.md
Outdated
@@ -87,7 +87,7 @@ The [list of active members](https://github.com/open-telemetry/opentelemetry-dot | |||
|
|||
Repo: [open-telemetry/opentelemetry-dotnet-instrumentation](https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation). | |||
|
|||
The [list of active members](https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation#community-roles) (both "approvers" and "maintainers") can be found in the [README.md](https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/blob/master/README.md#community-roles) file in the [repo](https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation). | |||
The [list of active members](https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation#community-roles) (both "approvers" and "maintainers") can be found in the [README.md](https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/tree/main/docs#community-roles) file in the [repo](https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation). |
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.
Should we also clarify that the README is under the docs folder in the repo?
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.
@nrcventura I don't know if we need to clarify that here; but I do think it would be helpful to specify that info in this place which would have helped me when I went searching for it: https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation#readme
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.
This feedback is addressed in open-telemetry/opentelemetry-dotnet-instrumentation#246.
Also updated the preceding link to point to the accurate place
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.
Thank you for noticing and fixing the links.
Before this change the link pointed to the top level readme which did not contain the community roles as indicated. Now the link points to the correct location where information can be found about membership and roles.