-
Notifications
You must be signed in to change notification settings - Fork 33
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
authorino version v0.18.0 #898
Conversation
Signed-off-by: Eguzki Astiz Lezaun <[email protected]>
Signed-off-by: Eguzki Astiz Lezaun <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #898 +/- ##
==========================================
+ Coverage 80.20% 81.21% +1.00%
==========================================
Files 64 89 +25
Lines 4492 6940 +2448
==========================================
+ Hits 3603 5636 +2033
- Misses 600 885 +285
- Partials 289 419 +130
Flags with carried forward coverage won't be shown. Click here to find out more.
|
@@ -72,6 +72,7 @@ require ( | |||
github.com/exponent-io/jsonpath v0.0.0-20210407135951-1de76d718b3f // indirect | |||
github.com/fatih/color v1.17.0 // indirect | |||
github.com/felixge/httpsnoop v1.0.4 // indirect | |||
github.com/foxcpp/go-mockdns v1.1.0 // indirect |
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.
It feels like this is not strictly related to bumping github.com/kuadrant/authorino
, but I guess it's OK?
github.com/tidwall/gjson v1.14.0 // indirect | ||
github.com/tidwall/match v1.1.1 // indirect | ||
github.com/tidwall/pretty v1.2.0 // indirect |
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.
These ones disappearing from the list of dependencies kinds of surprises me. Has Go become smarter on knowing this part of Authorino's code is not reached by the Kuadrant Operator maybe?
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.
blame go mod tidy
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.
OK. I'm gonna trust this is good news.
No description provided.