-
Notifications
You must be signed in to change notification settings - Fork 148
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
fix: lock protobufjs version #2569
Conversation
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Quality Gate passedIssues Measures |
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 ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2569 +/- ##
==========================================
- Coverage 84.42% 84.41% -0.01%
==========================================
Files 283 283
Lines 71021 71021
==========================================
- Hits 59956 59949 -7
- Misses 11065 11072 +7 ☔ View full report in Codecov by Sentry. |
* fix: lock protobufjs version Signed-off-by: Ivaylo Nikolov <[email protected]> * chore: update pnpm-lock in proto package Signed-off-by: Ivaylo Nikolov <[email protected]> --------- Signed-off-by: Ivaylo Nikolov <[email protected]> Signed-off-by: b-l-u-e <[email protected]>
* fix: lock protobufjs version Signed-off-by: Ivaylo Nikolov <[email protected]> * chore: update pnpm-lock in proto package Signed-off-by: Ivaylo Nikolov <[email protected]> --------- Signed-off-by: Ivaylo Nikolov <[email protected]> Signed-off-by: b-l-u-e <[email protected]>
Description:
Due to a bug in protobufjs version we need to lock
protobufjs
version so we can't groom some of our PRs. This PR updates the package.json and pnpm-lock.files. Deletes pnpm-lock file in the TCK folder due to having both npm-lock and pnpm-lock. Good practices say we should only have one lock file.Checklist