Skip to content

Conversation

@chalin
Copy link
Contributor

@chalin chalin commented Aug 29, 2024

/cc @cartermp @svrnm

@chalin chalin added the CI/infra Repo CI & infrastructure label Aug 29, 2024
@chalin chalin requested a review from a team August 29, 2024 15:48
@chalin
Copy link
Contributor Author

chalin commented Aug 29, 2024

/fix:format

@opentelemetrybot
Copy link
Contributor

@chalin
Copy link
Contributor Author

chalin commented Aug 29, 2024

I don't know if running a fix:* command over this PR will use the script in the PR (I'm thinking that it won't), but it's worth a try.

@chalin
Copy link
Contributor Author

chalin commented Aug 29, 2024

It doesn't seem like it. So we might just have to merge this PR and test it afterwards.

Copy link
Member

@tiffany76 tiffany76 left a comment

Choose a reason for hiding this comment

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

RSLGTM

@chalin chalin merged commit 470a5f8 into open-telemetry:main Aug 29, 2024
@chalin chalin deleted the chalin-im-tell-to-run-checks-2024-08-29 branch August 29, 2024 21:15
michael2893 pushed a commit to michael2893/opentelemetry.io that referenced this pull request Sep 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/infra Repo CI & infrastructure

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants