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: Upgrade OPA to v0.39.0 #1968
Conversation
Signed-off-by: Will Beason <willbeason@google.com>
The current version we're using is at the end of its support lifecycle. Also, 1.45 fully supports Go 1.18 for when we need to upgrade. Signed-off-by: Will Beason <willbeason@google.com>
Codecov Report
@@ Coverage Diff @@
## master #1968 +/- ##
==========================================
- Coverage 53.91% 53.78% -0.14%
==========================================
Files 103 103
Lines 9147 9147
==========================================
- Hits 4932 4920 -12
- Misses 3839 3848 +9
- Partials 376 379 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
do we want to hold this until v3.9? |
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
IMO if the new opa version does not introduce any issues, we should bump it now before releasing 3.8 since 3.9 will be awhile. WDYT? |
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
Upgrade OPA in frameworks library