Skip to content

Build with go 1.17#57

Merged
unguiculus merged 2 commits intocraftypath:mainfrom
andrei-radulescu:patch-1
Nov 2, 2021
Merged

Build with go 1.17#57
unguiculus merged 2 commits intocraftypath:mainfrom
andrei-radulescu:patch-1

Conversation

@andrei-radulescu
Copy link
Copy Markdown
Contributor

@unguiculus
Copy link
Copy Markdown
Member

Thanks, you will also have to update release.yaml and go.mod.

@andrei-radulescu
Copy link
Copy Markdown
Contributor Author

andrei-radulescu commented Nov 2, 2021

Thanks, you will also have to update release.yaml and go.mod.

done (although it would have worked like this as well - done it for mozilla sops) :) watch out for go.sum as go 1.17 states also all indirect requirements in the go.sum and the magefile needs an extra annotation for it to run under go 1.17

@unguiculus unguiculus changed the title Build with go 1.17 to reduce vulnerabilities Build with go 1.17 Nov 2, 2021
@unguiculus
Copy link
Copy Markdown
Member

I changed the PR title because the CVE is not a reason to update. The fix was back-ported to 1.16.9. Anyways, updating is a good idea 😉.

@unguiculus unguiculus merged commit a7ffaeb into craftypath:main Nov 2, 2021
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