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

Registry data cleanup: add urls to all authors #5608

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

svrnm
Copy link
Member

@svrnm svrnm commented Nov 14, 2024

fixes #5601

a few notes:

  • Everything in a repo https://github.com/open-telemetry/opentelemetry-* has "OpenTelemetry Authors", since the code is licensed and owned by the project. For the collector contrib repo we can later update this with the code owners, but that's not feasible right now
  • Where available I picked github handles, otherwise I looked up an URL or something. It's not 100% consistent but it wasn't consistent before that as well.

I removed the "warning" restriction for the authors.url authors.email with that.

@svrnm svrnm requested a review from a team as a code owner November 14, 2024 10:52
Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

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

@chalin chalin added this pull request to the merge queue Nov 15, 2024
Merged via the queue into open-telemetry:main with commit cd73780 Nov 15, 2024
18 checks passed
drewby pushed a commit to drewby/opentelemetry.io that referenced this pull request Nov 21, 2024
drewby pushed a commit to drewby/opentelemetry.io that referenced this pull request Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Registry data cleanup: update entries to eliminate warnings
2 participants