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

release: Update go modules for pre-release #1892

Conversation

stevenhorsman
Copy link
Member

We are working on the v0.9.0-alpha1 pre-release, so bump the go modules to use the new tag we will create once they are merged.

@@ -47,8 +47,8 @@ require (
github.com/aws/aws-sdk-go-v2/service/eks v1.29.5
github.com/aws/aws-sdk-go-v2/service/iam v1.22.5
github.com/aws/aws-sdk-go-v2/service/s3 v1.38.5
github.com/confidential-containers/cloud-api-adaptor/src/cloud-providers v0.8.2
github.com/confidential-containers/cloud-api-adaptor/src/peerpod-ctrl v0.8.2
github.com/confidential-containers/cloud-api-adaptor/src/cloud-providers v0.9.0-alpha0
Copy link
Member

Choose a reason for hiding this comment

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

This the "alpha" usage problem that you mentioned on last meeting? i.e., caa always tagged <version>-alphaN but now <version> is actually an alpha release too!

If so, would make sense to call it v0.9.0-alpha1-alpha0? I confess this seems ugly and I never saw such as versioning before...but just in case, let me ask :)

Copy link
Member Author

Choose a reason for hiding this comment

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

I considered that myself, but thought it was ugly. If it occured to you then it's probably worth switching :)

We are working on the `v0.9.0-alpha1` pre-release, so bump
the go modules to use the new tag we will create once they are merged.

Due to the pre-release of an alpha the version looks messy, but we will improve it soon

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
@stevenhorsman stevenhorsman force-pushed the 0.9.0-alpha1-pre-release-go-module-bump branch from d503d88 to dffbe03 Compare July 1, 2024 18:32
Copy link
Member

@wainersm wainersm left a comment

Choose a reason for hiding this comment

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

I hope never see those alpha of alpha again :)

Copy link
Member

@bpradipt bpradipt left a comment

Choose a reason for hiding this comment

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

/lgtm

@stevenhorsman stevenhorsman merged commit 68d6b93 into confidential-containers:main Jul 2, 2024
18 checks passed
@stevenhorsman stevenhorsman deleted the 0.9.0-alpha1-pre-release-go-module-bump branch July 2, 2024 08:34
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.

3 participants