-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Update dependencies (again) #2940
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2940 +/- ##
==========================================
- Coverage 76.96% 76.92% -0.05%
==========================================
Files 225 225
Lines 16893 16907 +14
==========================================
+ Hits 13001 13005 +4
- Misses 3061 3070 +9
- Partials 831 832 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@@ -44,14 +44,15 @@ require ( | |||
golang.org/x/term v0.5.0 | |||
golang.org/x/time v0.3.0 | |||
google.golang.org/grpc v1.53.0 | |||
google.golang.org/protobuf v1.28.1 | |||
google.golang.org/protobuf v1.28.2-0.20220831092852-f930b1dc76e8 |
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.
mmm...this is a not released version. Not optimal, now that we need to use protobuf for other things I hope it will not create issues.
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 didn't manually update this, it came from the github.com/jhump/protoreflect update, see this 🤷♂️
Decided to make another PR after #2931, because the xk6 tests still had some old
golang.org/x/
dependencies, and because the protoreflect library apparently had a pretty major release: https://github.com/jhump/protoreflect/releases/tag/v1.15.0