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

update to go 1.18 #2059

Merged
merged 1 commit into from
Jul 11, 2022
Merged

update to go 1.18 #2059

merged 1 commit into from
Jul 11, 2022

Conversation

asraa
Copy link
Contributor

@asraa asraa commented Jul 11, 2022

Signed-off-by: Asra Ali [email protected]

Summary

  • Bump to go 1.18
    This is a pre-requisite to updating rekor.
    Locally, ./hack/update-codegen.sh doesn't complete successfully for me, so this may not pass CI on the first run.

Release Note

Documentation

hectorj2f
hectorj2f previously approved these changes Jul 11, 2022
@hectorj2f hectorj2f dismissed their stale review July 11, 2022 16:25

Tests are failing! I missed that

@asraa
Copy link
Contributor Author

asraa commented Jul 11, 2022

Failing because go get isn't supported outside a module and the codegen scripts do this. working on it.

@asraa
Copy link
Contributor Author

asraa commented Jul 11, 2022

had to update knative.dev/hack to support go 1.18. an old version used go get, and that's no longer supported outside a module.

edit:
- uses: chainguard-dev/actions/goimports@84c993eaf02da1c325854fb272a4df9184bd80fc # main also does not support go.1.18

Signed-off-by: Asra Ali <[email protected]>

update

Signed-off-by: Asra Ali <[email protected]>

also update

Signed-off-by: Asra Ali <[email protected]>
Copy link
Contributor

@hectorj2f hectorj2f left a comment

Choose a reason for hiding this comment

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

lgtm

@dlorenc dlorenc merged commit a7c439a into sigstore:main Jul 11, 2022
@github-actions github-actions bot added this to the v1.10.0 milestone Jul 11, 2022
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