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

docs: add get_operation code snippets #12

Merged
merged 20 commits into from
Apr 8, 2022
Merged

docs: add get_operation code snippets #12

merged 20 commits into from
Apr 8, 2022

Conversation

changsongd
Copy link
Contributor

SnippetGen is currently only capable of generating simple snippets for service clients. The Operations related endpoints are not defined with the service proto. As a result, I will add this handwritten sample and reference it in the documentation.

@changsongd changsongd requested a review from a team as a code owner April 4, 2022 23:35
@changsongd changsongd requested a review from busunkim96 April 4, 2022 23:35
@snippet-bot
Copy link

snippet-bot bot commented Apr 4, 2022 •
edited
Loading

Here is the summary of possible violations 😱

There is a possible violation for not having product prefix.

The end of the violation section. All the stuff below is FYI purposes only.

Here is the summary of changes.

You are about to add 1 region tag.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: cloudoptimization Issues related to the googleapis/python-optimization API. labels Apr 4, 2022
@changsongd changsongd changed the title docs: Add get_operation code snippets docs: add get_operation code snippets Apr 4, 2022
@changsongd changsongd mentioned this pull request Apr 5, 2022
Closed
changsongd and others added 3 commits April 6, 2022 14:31
@changsongd @SurferJeffAtGoogle
@gcf-owl-bot @leahecole @parthea
docs: add sync api samples with json request (googleapis#13)
6bd0b03 
* add code snippets for sync and async api

* remove async test samples

* use f-string

* change project_id to input arg

* add noxfile

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* rename noxfile and add requirements

* rm noxfile

* rm noxfile local

* add root noxfile

* Update noxfile.py

* Update noxfile.py

* Update noxfile.py

Co-authored-by: Leah E. Cole <[email protected]>

* Apply suggestions from code review

* Update noxfile.py

* Update noxfile.py

* Update noxfile.py

* Update noxfile.py

* Update noxfile.py

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Add commit to trigger kokoro

* add indentation

* add type annotations

Co-authored-by: Jeffrey Rennie <[email protected]>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Leah E. Cole <[email protected]>
Co-authored-by: Anthonios Partheniou <[email protected]>
@changsongd
rebase and add type annotations
9c47fe5
@changsongd
fix operation_id type
93d64e9
@product-auto-label product-auto-label bot added size: l Pull request size is large. and removed size: m Pull request size is medium. labels Apr 6, 2022
@product-auto-label product-auto-label bot added size: m Pull request size is medium. and removed size: l Pull request size is large. labels Apr 6, 2022
busunkim96
busunkim96 reviewed Apr 6, 2022
View reviewed changes
@changsongd changsongd requested a review from busunkim96 April 7, 2022 15:56
@changsongd
Copy link
Contributor Author

Here is the summary of possible violations 😱

There is a possible violation for not having product prefix.
The end of the violation section. All the stuff below is FYI purposes only.

Here is the summary of changes.

You are about to add 1 region tag.
This comment is generated by snippet-bot. If you find problems with this result, please file an issue at: https://github.com/googleapis/repo-automation-bots/issues. To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

snippet-bot is flagging that the product is not yet public.

googleapis/repo-automation-bots#3182

busunkim96
busunkim96 reviewed Apr 8, 2022
View reviewed changes
@busunkim96 busunkim96 added the owlbot:run Add this label to trigger the Owlbot post processor. label Apr 8, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Apr 8, 2022
@busunkim96 busunkim96 merged commit a95c19f into googleapis:main Apr 8, 2022
@release-please release-please bot mentioned this pull request Apr 8, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@busunkim96 busunkim96 busunkim96 approved these changes

@xiliucity xiliucity xiliucity approved these changes

Assignees
No one assigned
Labels
api: cloudoptimization Issues related to the googleapis/python-optimization API. size: m Pull request size is medium.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@changsongd @busunkim96 @xiliucity @SurferJeffAtGoogle