Skip to content
This repository was archived by the owner on Nov 18, 2025. It is now read-only.

fix: use the full type name for ErrorInfo#1093

Merged
summer-ji-eng merged 1 commit intogoogleapis:masterfrom
idan-at:patch-2
Aug 31, 2021
Merged

fix: use the full type name for ErrorInfo#1093
summer-ji-eng merged 1 commit intogoogleapis:masterfrom
idan-at:patch-2

Conversation

@idan-at
Copy link
Contributor

@idan-at idan-at commented Aug 31, 2021

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes googleapis/google-cloud-node-core#359 🦕

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 31, 2021
@idan-at idan-at marked this pull request as ready for review August 31, 2021 14:16
@idan-at idan-at requested a review from a team as a code owner August 31, 2021 14:16
@summer-ji-eng
Copy link
Contributor

Hello @idan-at , thank you sent the pull request. Try to understand what error or issue you met for non-full type error info?

@idan-at
Copy link
Contributor Author

idan-at commented Aug 31, 2021

@summer-ji-eng see googleapis/google-cloud-node-core#359 for a detailed explanation.

unrelated, can you please explain why aren’t the tests running? Is it something I missed?

@summer-ji-eng
Copy link
Contributor

@idan-at I see the problem. I am fixing it now. I should sent out the PR by EOD.
Regarding to the test, it require an approval for publish contributor. Let me double check.

@summer-ji-eng summer-ji-eng added the kokoro:run Add this label to force Kokoro to re-run the tests. label Aug 31, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:run Add this label to force Kokoro to re-run the tests. label Aug 31, 2021
@summer-ji-eng summer-ji-eng added kokoro:run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. and removed kokoro:run Add this label to force Kokoro to re-run the tests. labels Aug 31, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 31, 2021
@summer-ji-eng summer-ji-eng merged commit c1e646a into googleapis:master Aug 31, 2021
gcf-merge-on-green bot pushed a commit that referenced this pull request Aug 31, 2021
@idan-at idan-at deleted the patch-2 branch September 1, 2021 10:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement. kokoro:run Add this label to force Kokoro to re-run the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

promote ErrorInfo feature fails in runtime when using a stricter version of protobufJs

3 participants