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

docs: remove migrated samples #180

Merged
dizcology merged 2 commits into from
Sep 19, 2023
Merged

docs: remove migrated samples #180

dizcology merged 2 commits into from
Sep 19, 2023

Conversation

@dizcology
Copy link
Contributor

@dizcology dizcology commented Sep 11, 2023

@dizcology dizcology requested a review from a team as a code owner September 11, 2023 22:37
@snippet-bot
Copy link

snippet-bot bot commented Sep 11, 2023 •
edited
Loading

Here is the summary of changes.

You are about to delete 26 region tags.

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: xl Pull request size is extra large. api: videostitcher Issues related to the googleapis/python-video-stitcher API. samples Issues that are directly related to samples. labels Sep 11, 2023
@dizcology dizcology added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Sep 11, 2023
@dizcology
Copy link
Contributor Author

dizcology commented Sep 11, 2023

Merge only after documentation pages have been updated.
@dizcology
Copy link
Contributor Author

dizcology commented Sep 15, 2023

cloud.google.com/video-stitcher/docs documentation pages had already been updated to use the migrated samples at https://github.com/GoogleCloudPlatform/python-docs-samples/tree/main/video/stitcher. Example: https://cloud.google.com/video-stitcher/docs/samples/videostitcher-list-slates?hl=en#videostitcher_list_slates-python

This clean up PR is ready to be merged.
@dizcology dizcology removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Sep 15, 2023
@dizcology dizcology merged commit 73af7e9 into main Sep 19, 2023
@dizcology dizcology deleted the remove-migrated-samples branch September 19, 2023 16:11
@release-please release-please bot mentioned this pull request Sep 13, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: videostitcher Issues related to the googleapis/python-video-stitcher API. samples Issues that are directly related to samples. size: xl Pull request size is extra large.

2 participants

@dizcology @m-strzelczyk