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

Bump default Go version to 1.19 #3694

Merged
merged 3 commits into from
Feb 8, 2023
Merged

Conversation

MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Feb 7, 2023

The latest golangci-lint version requires Go >= 1.19: #3690

We are adding support for Go 1.20 in #3693, this preemptively bumps the minimum version support of the default Go prior to dropping Go 1.18 after the next release. We already have done this in contrib as it is needed for CI compatibility there as well.

@MrAlias MrAlias added pkg:tooling Related to the tooling package Skip Changelog PRs that do not require a CHANGELOG.md entry labels Feb 7, 2023
Copy link
Member

@XSAM XSAM left a comment

Choose a reason for hiding this comment

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

lgtm

@codecov
Copy link

codecov bot commented Feb 8, 2023

Codecov Report

Merging #3694 (093a04b) into main (a7fb378) will decrease coverage by 0.1%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #3694     +/-   ##
=======================================
- Coverage   79.7%   79.7%   -0.1%     
=======================================
  Files        171     171             
  Lines      12673   12673             
=======================================
- Hits       10105   10101      -4     
- Misses      2355    2359      +4     
  Partials     213     213             
Impacted Files Coverage Δ
sdk/trace/batch_span_processor.go 80.2% <0.0%> (-1.8%) ⬇️

@MrAlias MrAlias merged commit fb5c599 into open-telemetry:main Feb 8, 2023
@MrAlias MrAlias deleted the min-go-1.19 branch February 8, 2023 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg:tooling Related to the tooling package Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants