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

build: update to Go 1.18 #154

Merged
merged 1 commit into from Dec 15, 2022
Merged

Conversation

lucacome
Copy link
Contributor

Bumps go to 1.18 for v1

@shizhMSFT
Copy link
Contributor

shizhMSFT commented May 26, 2022

Since this is library code, adopters may still use older golang versions. We need to be careful on bumping up go versions.

@jdolitsky @deitch Do you have any inputs since it is for v1?

@shizhMSFT shizhMSFT added the v1 Things belongs to version 1.x label May 26, 2022
@shizhMSFT shizhMSFT self-requested a review May 27, 2022 10:03
@TerryHowe
Copy link
Member

Close this maybe?

@shizhMSFT
Copy link
Contributor

Close this maybe?

We plan to close it after releasing oras-go v2.0.0.

@lucacome lucacome force-pushed the update-go-1.18-v1 branch 2 times, most recently from e02ba92 to 1539f9f Compare December 10, 2022 00:49
@lucacome
Copy link
Contributor Author

We plan to close it after releasing oras-go v2.0.0.

I've updated the PR, but re-reading this now I'm not sure if the plan is to merge it or just close it 😅

@shizhMSFT shizhMSFT changed the title Update to Go 1.18 build: update to Go 1.18 Dec 15, 2022
Signed-off-by: Luca Comellini <luca.com@gmail.com>
Copy link
Contributor

@shizhMSFT shizhMSFT left a comment

Choose a reason for hiding this comment

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

LGTM

@shizhMSFT shizhMSFT merged commit 0a083d7 into oras-project:v1 Dec 15, 2022
@lucacome lucacome deleted the update-go-1.18-v1 branch December 15, 2022 00:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v1 Things belongs to version 1.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants