Skip to content

Add grpc-web test and update grpc-web dependency version.#12

Merged
gonzojive merged 9 commits intomainfrom
grpc-web-test
Oct 7, 2023
Merged

Add grpc-web test and update grpc-web dependency version.#12
gonzojive merged 9 commits intomainfrom
grpc-web-test

Conversation

@gonzojive
Copy link
Copy Markdown
Owner

@gonzojive gonzojive commented Oct 7, 2023

  1. Update repository dependency to point to grpc/grpc-web. Update ES6 .d.ts imports with comment about corresponding proto import path grpc/grpc-web#1330 was merged, so there's no longer a need to depend on a fork.
  2. Add a test of the grpc-web functionality and require the grpc-web node module.
  3. Ensure all rewritten imports include the ".mjs" suffix. It was previously omitted for some imports.

@gonzojive gonzojive changed the title Remove unused proto_plugin targets. Add grpc-web test and update grpc-web dependency version. Oct 7, 2023
@gonzojive gonzojive merged commit f3a77e3 into main Oct 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant