Skip to content
This repository has been archived by the owner on Jul 20, 2023. It is now read-only.

fix: run synthtool hack to compile code #6

Merged
merged 3 commits into from
Aug 3, 2020
Merged

fix: run synthtool hack to compile code #6

merged 3 commits into from
Aug 3, 2020

Conversation

sofisl
Copy link
Contributor

@sofisl sofisl commented Jul 27, 2020

No description provided.

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jul 27, 2020
@codecov
Copy link

codecov bot commented Jul 27, 2020

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   97.17%   97.17%           
=======================================
  Files           5        5           
  Lines        1841     1841           
  Branches       83       83           
=======================================
  Hits         1789     1789           
  Misses         51       51           
  Partials        1        1           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 009e512...cf00b44. Read the comment docs.

Comment on lines +32 to +33
s.replace('src/v1/cloud_functions_service_client.ts', "function:string", "function_: string")
s.replace('src/v1/cloud_functions_service_client.ts', "function: function", "function: function_")

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we plan to have this in perpetuity or is this something the generator should be able to handle? For any substantive synthtool replacements, we should have a bug filed on the upstream generator or team (and comment here with the link) so that these clients can generate without synthtool hacks.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes! Alex F. is working on it. We discussed it in our Node meeting yesterday. He asked me to implement this hack in the meantime, as it might take some weeks to work.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sofisl sofisl added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. and removed kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. labels Jul 30, 2020
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Jul 31, 2020
@sofisl sofisl merged commit 2b080c5 into master Aug 3, 2020
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants