Skip to content

Conversation

@fredbi
Copy link
Member

@fredbi fredbi commented Sep 10, 2025

  • upgraded go-opentelemetry packages

Change type

Please select: 🆕 New feature or enhancement|🔧 Bug fix'|📃 Documentation update

go compiler version and dependencies upgrade

Short description

moved up from a go1.20 requirement to a more modern go1.24 (almost a 2 years lag!).
this allowed to update OTEL packages.

Fixes

Full description

Checklist

  • I have signed all my commits with my name and email (see DCO. This does not require a PGP-signed commit
  • I have rebased and squashed my work, so only one commit remains
  • I have added tests to cover my changes.
  • I have properly enriched go doc comments in code.
  • I have properly documented any breaking change.

* upgraded go-opentelemetry packages

Signed-off-by: Frederic BIDON <[email protected]>
@codecov
Copy link

codecov bot commented Sep 10, 2025

Codecov Report

❌ Patch coverage is 69.23077% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.84%. Comparing base (75ee09a) to head (99f4735).
⚠️ Report is 6 commits behind head on master.

Files with missing lines Patch % Lines
middleware/context.go 33.33% 4 Missing ⚠️

❌ Your patch status has failed because the patch coverage (69.23%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #362   +/-   ##
=======================================
  Coverage   81.83%   81.84%           
=======================================
  Files          49       49           
  Lines        4074     4076    +2     
=======================================
+ Hits         3334     3336    +2     
  Misses        628      628           
  Partials      112      112           
Flag Coverage Δ
1.20-macos-latest ?
1.20-ubuntu-latest ?
1.20-windows-latest ?
oldstable-macos-latest 81.84% <69.23%> (+<0.01%) ⬆️
oldstable-ubuntu-latest 81.84% <69.23%> (+<0.01%) ⬆️
oldstable-windows-latest 81.84% <69.23%> (+<0.01%) ⬆️
stable-macos-latest 81.84% <69.23%> (+<0.01%) ⬆️
stable-ubuntu-latest 81.84% <69.23%> (+<0.01%) ⬆️
stable-windows-latest 81.84% <69.23%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@fredbi fredbi merged commit 0628cdf into go-openapi:master Sep 10, 2025
10 of 11 checks passed
@fredbi fredbi deleted the chore/upgrade-go124 branch September 10, 2025 19:50
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.

1 participant