Skip to content

Revert "chore: upgrade texttospeech samples to new surface" #1902

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 31, 2023

Conversation

yash30201
Copy link
Contributor

Reverts #1880

As this need not to be merged until the new surface is GA.

More details at #1886

@yash30201 yash30201 requested a review from bshaffer as a code owner July 31, 2023 06:39
@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Jul 31, 2023
@yash30201 yash30201 merged commit fa6f3a4 into main Jul 31, 2023
@yash30201 yash30201 deleted the revert-1880-texttospeech-v2gapic branch July 31, 2023 06:45
pattishin pushed a commit that referenced this pull request Aug 1, 2023
pattishin pushed a commit that referenced this pull request Aug 1, 2023
pattishin pushed a commit that referenced this pull request Aug 2, 2023
pattishin added a commit that referenced this pull request Aug 25, 2023
* feat: initial commit of run mc php nginx sample

* chore: upgrade texttospeech samples to new surface (#1880)

* Revert "chore: upgrade texttospeech samples to new surface (#1880)" (#1902)

This reverts commit 18c6872.

* test: update php mc test and check deployment logs

* test: updating test and clean up

* chore: clean up

* refactor: rename test

* Update run/multi-container/hello-php-nginx-sample/nginx/nginx.conf

Co-authored-by: Julien Breux <[email protected]>

* Update run/multi-container/hello-php-nginx-sample/Makefile

Co-authored-by: Julien Breux <[email protected]>

* fix: flip container deps

* Update run/multi-container/hello-php-nginx-sample/php-app/Dockerfile

Co-authored-by: Adam Ross <[email protected]>

* Update run/multi-container/hello-php-nginx-sample/php-app/opcache.ini

Co-authored-by: Adam Ross <[email protected]>

* Update run/multi-container/hello-php-nginx-sample/README.md

Co-authored-by: Adam Ross <[email protected]>

* refactor: updating with suggestions (updating readme, Cloud Run memory limit updates, rm makefile, add composer)

* fix: resolve snippet-bot failures about lack of product name

* refactor: update .ini and readme with concurrency/memory info

* chore: clean up readme

* refactor: updating comment about dockerfile context

---------

Co-authored-by: Brent Shaffer <[email protected]>
Co-authored-by: Yash Sahu <[email protected]>
Co-authored-by: Julien Breux <[email protected]>
Co-authored-by: Adam Ross <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant