Skip to content

Document that Patch releases of Go may be adopted#15052

Open
ArthurSens wants to merge 1 commit intoopen-telemetry:mainfrom
ArthurSens:min-go-version
Open

Document that Patch releases of Go may be adopted#15052
ArthurSens wants to merge 1 commit intoopen-telemetry:mainfrom
ArthurSens:min-go-version

Conversation

@ArthurSens
Copy link
Copy Markdown
Member

Description

This PR adjusts our internal documentation to clarify that we allow upgrading the minimum Go version in go.mod files, as long as it is patch releases. We still require following the N-1 criteria, where N is the latest minor version of Go.

This unblocks upgrades of quite a few dependencies:

Link to tracking issue

Fixes open-telemetry/opentelemetry-collector-contrib#46609

Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.24%. Comparing base (76ede07) to head (67e845b).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #15052      +/-   ##
==========================================
- Coverage   91.24%   91.24%   -0.01%     
==========================================
  Files         697      697              
  Lines       44647    44647              
==========================================
- Hits        40738    40736       -2     
- Misses       2769     2770       +1     
- Partials     1140     1141       +1     

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

Copy link
Copy Markdown
Contributor

@axw axw left a comment

Choose a reason for hiding this comment

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

LGTM. IMO this strikes a better balance between compatibility and security, and it's a reasonably standard thing to do in the Go ecosystem.

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.

bump minimum version to go 1.25.7

4 participants