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

Add the tidying back for otelcontribcol but only on the release thingie #35609

Merged
merged 1 commit into from
Oct 4, 2024

Conversation

mx-psi
Copy link
Member

@mx-psi mx-psi commented Oct 4, 2024

Description

The change in #35608 was too aggressive, this reverts the changes in release-prepare-release.sh

@mx-psi mx-psi requested a review from a team as a code owner October 4, 2024 11:35
@mx-psi mx-psi requested a review from andrzej-stencel October 4, 2024 11:35
@mx-psi mx-psi added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Oct 4, 2024
@mx-psi
Copy link
Member Author

mx-psi commented Oct 4, 2024

Okay, I didn't realize before but this can be tested in my fork, here is a test run https://github.com/mx-psi/opentelemetry-collector-contrib/actions/runs/11179235444

@mx-psi
Copy link
Member Author

mx-psi commented Oct 4, 2024

Okay, the action pushed the branch and it failed to open the PR (which is to be expected given that I don't have a github token in my fork). Merging

@mx-psi mx-psi merged commit 0cbfcfc into open-telemetry:main Oct 4, 2024
162 of 163 checks passed
@mx-psi mx-psi deleted the mx-psi/fix-part-2 branch October 4, 2024 11:56
@github-actions github-actions bot added this to the next release milestone Oct 4, 2024
jriguera pushed a commit to springernature/opentelemetry-collector-contrib that referenced this pull request Oct 4, 2024
…ie (open-telemetry#35609)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

The change in open-telemetry#35608 was too aggressive, this reverts the changes in
release-prepare-release.sh
ghost pushed a commit to sematext/opentelemetry-collector-contrib that referenced this pull request Oct 9, 2024
…ie (open-telemetry#35609)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

The change in open-telemetry#35608 was too aggressive, this reverts the changes in
release-prepare-release.sh
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants