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 codeowners to a single place #1784

Merged

Conversation

cijothomas
Copy link
Member

Since non-spec components are already moved to contrib repo, we don't have the need to maintain per component owners file. This PR remove all codeowners file in individual components.
Also moves the main codeowners from root to ./github folder, which is recognized by Github. This makes the top level directly leaner.

@cijothomas cijothomas requested a review from a team as a code owner May 19, 2024 04:06
Copy link

codecov bot commented May 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.1%. Comparing base (2210d7b) to head (456c053).

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1784   +/-   ##
=====================================
  Coverage   74.1%   74.1%           
=====================================
  Files        125     125           
  Lines      19484   19484           
=====================================
+ Hits       14451   14452    +1     
+ Misses      5033    5032    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cijothomas cijothomas merged commit 29ac005 into open-telemetry:main May 19, 2024
21 checks passed
@cijothomas cijothomas deleted the cijothomas/cleanup-codeowners branch May 19, 2024 05:16
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.

None yet

2 participants