-
Notifications
You must be signed in to change notification settings - Fork 479
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
feat: refactor protobufs and utils to remove grpc-js dependency #802
Conversation
🦋 Changeset detectedLatest commit: a043e15 The changes in this PR will be included in the next version bump. This PR includes changesets to release 5 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
New dependency changes detected. Learn more about Socket for GitHub ↗︎ 👍 No new dependency issues detected in pull request Bot CommandsTo ignore an alert, reply with a comment starting with Pull request alert summary
📊 Modified Dependency Overview:
|
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #802 +/- ##
==========================================
- Coverage 73.03% 72.48% -0.56%
==========================================
Files 61 62 +1
Lines 5359 5379 +20
Branches 1216 1212 -4
==========================================
- Hits 3914 3899 -15
- Misses 1351 1380 +29
- Partials 94 100 +6
... and 2 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
9af4c4a
to
7810d91
Compare
Motivation
Part of #573, removed grpc-js dependency from protobufs (thus utils).
Change Summary
Merge Checklist
Choose all relevant options below by adding an
x
now or at any time before submitting for reviewAdditional Context