Skip to content

Fix node interop build scripts#19006

Merged
murgatroid99 merged 1 commit intogrpc:masterfrom
murgatroid99:node_interop_build_fix
May 10, 2019
Merged

Fix node interop build scripts#19006
murgatroid99 merged 1 commit intogrpc:masterfrom
murgatroid99:node_interop_build_fix

Conversation

@murgatroid99
Copy link
Copy Markdown
Member

This fixes #15464 in combination with grpc/grpc-node#862.

@murgatroid99 murgatroid99 added the release notes: no Indicates if PR should not be in release notes label May 10, 2019
@murgatroid99
Copy link
Copy Markdown
Member Author

Re-running tests with grpc/grpc-node#862 merged.

@murgatroid99
Copy link
Copy Markdown
Member Author

Trying again with grpc/grpc-node#865 merged.

@murgatroid99
Copy link
Copy Markdown
Member Author

The interop failures are now test failures instead of build failures, which is a success for this PR. I'm just merging it.

@murgatroid99 murgatroid99 merged commit c2716c6 into grpc:master May 10, 2019
@srini100
Copy link
Copy Markdown
Contributor

@murgatroid99, is this needed in v1.21.x? If yes, can you please merge?

@nicolasnoble
Copy link
Copy Markdown
Contributor

@srini100 not really, no.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area/interop kind/bug lang/node release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

node and nodepurejs interop failing to build

4 participants