-
Notifications
You must be signed in to change notification settings - Fork 327
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
chore(*) versions bump and CI go cache #1409
Conversation
Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
e5f133e
to
75955ea
Compare
Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
75955ea
to
6ba26b2
Compare
@@ -36,3 +36,6 @@ issues: | |||
- linters: | |||
- staticcheck | |||
text: "SA1019: l.UseOriginalDst is deprecated: Do not use." # TODO What is the up-to-date alternative ? | |||
- linters: | |||
- staticcheck | |||
text: "c.Http2ProtocolOptions is deprecated: Do not use." # TODO What is the up-to-date alternative ? |
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.
I was pretty sure that comment in the code that ignores it was enough. Did something change with a new version?
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.
Yep the messages did appear on my computer so I had to disable it.
@@ -220,6 +212,7 @@ jobs: | |||
|
|||
check: | |||
executor: golang | |||
resource_class: large |
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 real reason for this PR as far as I understand is this line because golanglint-ci has not enough memory.
We need to add this reason either in PR or here as a little comment (I prefer the latter)
Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
* chore(*) versions bump * test(*) ci cache optimization Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com> (cherry picked from commit d89548d)
* chore(*) versions bump * test(*) ci cache optimization Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
Summary
Fixing the falling CI check and bumping versions meanwhile:
make vet