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 golang from 1.18 to 1.20 #616

Merged
merged 1 commit into from
Jan 30, 2024
Merged

Bump golang from 1.18 to 1.20 #616

merged 1 commit into from
Jan 30, 2024

Conversation

krmichelos
Copy link
Contributor

@krmichelos krmichelos commented Nov 1, 2023

Golang 1.20 is the oldest version that is supported

@krmichelos krmichelos marked this pull request as ready for review November 1, 2023 13:53
@krmichelos
Copy link
Contributor Author

@estahn, do you need anything from me?

Copy link

codecov bot commented Nov 19, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (123ee5b) 34.08% compared to head (31f9a68) 34.08%.
Report is 68 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #616   +/-   ##
=======================================
  Coverage   34.08%   34.08%           
=======================================
  Files           9        9           
  Lines         848      848           
=======================================
  Hits          289      289           
  Misses        534      534           
  Partials       25       25           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@krmichelos
Copy link
Contributor Author

krmichelos commented Nov 21, 2023

I am mostly offline until after Thanksgiving, but I will take a look at the failures when I am back if it is still an issue.

@estahn
Copy link
Owner

estahn commented Nov 21, 2023

@krmichelos Thanks. I'm travelling over the next couple of weeks, so have limited time myself.

@krmichelos
Copy link
Contributor Author

@estahn I believe I have all the lint related issues resolved

@krmichelos
Copy link
Contributor Author

@estahn, I know it is a busy time of year, but would you be able to approve the tests to run so I can see if they pass?

@krmichelos
Copy link
Contributor Author

I will be mostly unavailable until after the new year

@estahn estahn merged commit 93b32c2 into estahn:main Jan 30, 2024
14 checks passed
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.

2 participants