Skip to content
This repository has been archived by the owner on Dec 16, 2021. It is now read-only.

chore: update grpc #392

Merged
merged 19 commits into from
Aug 11, 2021
Merged

chore: update grpc #392

merged 19 commits into from
Aug 11, 2021

Conversation

strophy
Copy link
Contributor

@strophy strophy commented Aug 3, 2021

Issue being fixed or feature implemented

The deprecated grpc dependency needed updating to @grpc/grpc-js for future support and to speed up builds in CI.

What was done?

  • Remove obsolete grpc
  • Add @grpc/grpc-js
  • Update and dedupe deps

How Has This Been Tested?

  • With npm run test
  • In CI

Breaking Changes

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@strophy strophy added this to the v0.21.0 milestone Aug 11, 2021
@strophy strophy requested a review from antouhou August 11, 2021 09:07
@strophy strophy marked this pull request as ready for review August 11, 2021 09:07
@strophy strophy merged commit 7049371 into v0.21-dev Aug 11, 2021
@strophy strophy deleted the update-grpc branch August 11, 2021 10:34
@strophy strophy restored the update-grpc branch August 11, 2021 10:41
@strophy strophy deleted the update-grpc branch September 26, 2021 11:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants