opentelemetry-collector
opentelemetry-collector copied to clipboard
[chore] validate version format to prepare-release workflow
Description: <Describe what has changed.> enhance the "Automation - Prepare Release" GitHub Actions workflow by adding version format validation. The workflow now includes a new job, "validate-version," which checks whether the provided version inputs match the expected schema
Link to tracking Issue: <Issue number if applicable> #7627
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 90.60%. Comparing base (
cc88aee) to head (a16d3e7). Report is 52 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #9409 +/- ##
==========================================
+ Coverage 90.58% 90.60% +0.01%
==========================================
Files 347 347
Lines 18247 18247
==========================================
+ Hits 16529 16532 +3
+ Misses 1388 1386 -2
+ Partials 330 329 -1
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@TylerHelmuth the review is stale from last week, could you please review the PR. I think I have covered all the aspects, do let me know if any further changes required, regards, Sanket
@anuraaga @TylerHelmuth if this is good enough will you please consider this for merging, it's been stale from the last 2 weeks.
Regards, Sanket
@open-telemetry/collector-maintainers please take a look.
Will test this on Monday's release :)
Looks like there is some sort of issue: https://github.com/open-telemetry/opentelemetry-collector/actions/runs/8110269766/workflow @Sanket-0510 could you have a look?
Looks like there is some sort of issue: https://github.com/open-telemetry/opentelemetry-collector/actions/runs/8110269766/workflow @Sanket-0510 could you have a look?
there is an intendation issue i guess https://github.com/open-telemetry/opentelemetry-collector/blob/94b5951a3ec9526765bd631728d4f7fe12fad6bd/.github/workflows/prepare-release.yml#L27C1-L32C1
@mx-psi I will open a PR to fix this.