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

Move semconv schema definition from build tools #307

Merged
merged 4 commits into from
Aug 11, 2024

Conversation

.github/CODEOWNERS Outdated Show resolved Hide resolved
Copy link
Contributor

@lquerel lquerel left a comment

Choose a reason for hiding this comment

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

I left a comment regarding CODEOWNERS.

@lquerel
Copy link
Contributor

lquerel commented Aug 9, 2024

@lmolkova LGTM but I'd like to get @jsuereth feedback on this one to double the CODEOWNERS changes.

By the way, I am not listed as a maintainer in this file: https://github.com/open-telemetry/community/blob/85f75d75789938ddfb42a9e3f251b85461cb210a/config.yaml. I'm not sure of the implications.

@lquerel lquerel self-requested a review August 9, 2024 21:57
@jsuereth
Copy link
Contributor

jsuereth commented Aug 11, 2024

Reagarding: https://github.com/open-telemetry/community/blob/85f75d75789938ddfb42a9e3f251b85461cb210a/config.yaml. We're moving to an git-ops like repository setup - [edit] I'm wondering if the current config in Github looks like that...

@jsuereth jsuereth merged commit 31f7f52 into open-telemetry:main Aug 11, 2024
20 checks passed
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.

Move yaml schema definition to semantic conventions
3 participants