Skip to content

Conversation

paulojmdias
Copy link
Member

@paulojmdias paulojmdias commented Sep 25, 2025

Description

This PR ports the check-codeowners workflow from the opentelemetry-collector-contrib repository.

On this side, the command referenced in the workflow is make generate-codeowners instead of make gencodeowners as is in the contrib version. In the Makefile, I added the -skipgithub to avoid giving an error like below when the members run it.

2025/09/25 22:52:36 set the environment variable `GITHUB_TOKEN` to a PAT token to authenticate

I ran gengithub, which also produced some changes in issue templates.

Like the PR #13898, I also removed the make install-tools from this workflow.

The idea to port this workflow here came from the PR open-telemetry/opentelemetry-collector-contrib#42999 discussion in Slack.

@paulojmdias paulojmdias requested a review from a team as a code owner September 25, 2025 22:03
@github-actions github-actions bot added the release:risky-change This change may affect the release label Sep 25, 2025
Copy link

codecov bot commented Sep 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.68%. Comparing base (0694086) to head (ee74366).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13908      +/-   ##
==========================================
+ Coverage   91.66%   91.68%   +0.01%     
==========================================
  Files         652      652              
  Lines       42506    42506              
==========================================
+ Hits        38964    38971       +7     
+ Misses       2734     2729       -5     
+ Partials      808      806       -2     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:risky-change This change may affect the release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants