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
build: Update Gatekeeper with frameworks interface changes #1845
build: Update Gatekeeper with frameworks interface changes #1845
Conversation
Also, enforce using a consistent import for the frameworks Client package. This ensures automated changes are easier, and ensures the Client type is referenced consistently throughout the code (instead of being confused with the k8s API client. Signed-off-by: Will Beason <willbeason@google.com>
The big thing here is that the Client is easier to instantiate, so all of the places we do this are now smaller. |
It looks like this is failing due to lint errors from a linter added by this PR? |
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.
LGTM after lint fix
Looks like there's a diff in go mod |
Signed-off-by: Will Beason <willbeason@google.com>
6475320
to
709762d
Compare
Signed-off-by: Will Beason <willbeason@google.com>
Signed-off-by: Will Beason <willbeason@google.com>
Signed-off-by: Will Beason <willbeason@google.com>
@sozercan Fixed |
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.
LGTM
Codecov Report
@@ Coverage Diff @@
## master #1845 +/- ##
==========================================
+ Coverage 52.12% 52.21% +0.09%
==========================================
Files 100 100
Lines 8956 8952 -4
==========================================
+ Hits 4668 4674 +6
+ Misses 3911 3903 -8
+ Partials 377 375 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Also, enforce using a consistent import for the frameworks Client
package. This ensures automated changes are easier, and ensures the
Client type is referenced consistently throughout the code (instead of
being confused with the k8s API client.
Signed-off-by: Will Beason willbeason@google.com