-
Notifications
You must be signed in to change notification settings - Fork 617
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 Go version to 1.19.1 #3398
Conversation
pickup the fix for https://nvd.nist.gov/vuln/detail/CVE-2022-27664
6444527
to
c060331
Compare
There are a lot of whitespace changes -- can you update the description to explain how these ended up in the change set? |
yes, updated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The io/ioutil
to os
changes are awesome! Thanks for making those!
* Bump Go version to 1.19.1 pickup the fix for https://nvd.nist.gov/vuln/detail/CVE-2022-27664 * make goimports * make goimports -f init_Makefile * fix seelog xml * deprecate ioutil
Pickup the fix for https://nvd.nist.gov/vuln/detail/CVE-2022-27664
Most of this is whitespace and comment changes, which are due to changes in go 1.19 go fmt, which now formats comments for the sake of the documentation, see https://tip.golang.org/doc/go1.19#go-doc
Testing
PR tests
New tests cover the changes: no
Description for the changelog
Bugfix: Bump Go to 1.19.1 for CVE-2022-27664
Licensing
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.