Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

HemangChothani
Copy link
Contributor

@HemangChothani HemangChothani commented Apr 9, 2020

Fixes #50

Hope this Draft PR helps to fix issue quickly.

  • This draft PR includes few gapic .rst files changes which i can't address so can't open Public PR, but changed that gapic .rst files for just references.

@HemangChothani HemangChothani requested a review from larkee April 9, 2020 13:12
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 9, 2020
@HemangChothani
Copy link
Contributor Author

@larkee Please let me know could i make changes for docs/gapic/*.rst files manually?

@larkee
Copy link
Contributor

larkee commented Apr 14, 2020

Good news! Those files aren't generated so this PR looks good to me :)

I understand the confusion. I had to run the synthtool to confirm. I suspect the reason these files are in folder named gapic is to match the folder structure of the files they're generating documentation for.

@HemangChothani HemangChothani marked this pull request as ready for review April 14, 2020 06:23
Copy link
Contributor

@larkee larkee left a comment

Choose a reason for hiding this comment

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

Thank you for handling this!

@larkee larkee merged commit c45f4fc into googleapis:master Apr 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Investigate new Sphinx release changes
3 participants