Skip to content

[chore] Use Make target for githubgen invocation#46151

Merged
evan-bradley merged 2 commits into
open-telemetry:mainfrom
evan-bradley:make-codeowners
Feb 17, 2026
Merged

[chore] Use Make target for githubgen invocation#46151
evan-bradley merged 2 commits into
open-telemetry:mainfrom
evan-bradley:make-codeowners

Conversation

@evan-bradley
Copy link
Copy Markdown
Contributor

Description

Follow up from #45739 to use Make to invoke githubgen.

I tried to clean things up a bit, but wasn't sure what should be used when and what makes sense to change. I've opened an issue to track this in case anyone would like to go deeper: #46150.

@evan-bradley evan-bradley requested a review from a team as a code owner February 17, 2026 15:15
@evan-bradley evan-bradley requested a review from mwear February 17, 2026 15:15
Copy link
Copy Markdown
Member

@mx-psi mx-psi left a comment

Choose a reason for hiding this comment

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

I don't understand the CI failure, this seems reasonable and I think we should merge it

@evan-bradley
Copy link
Copy Markdown
Contributor Author

I don't understand it either, but it appeared to be a simple fix, so I gave it a shot.

@evan-bradley evan-bradley merged commit 3ace8e8 into open-telemetry:main Feb 17, 2026
189 checks passed
geekdave pushed a commit to oso-team/opentelemetry-collector-contrib that referenced this pull request Feb 20, 2026
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

Follow up from
open-telemetry#45739
to use Make to invoke githubgen.

I tried to clean things up a bit, but wasn't sure what should be used
when and what makes sense to change. I've opened an issue to track this
in case anyone would like to go deeper:
open-telemetry#46150.
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.

6 participants