Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Updated README.md#759

Merged
theacodes merged 1 commit into
GoogleCloudPlatform:masterGoogleCloudPlatform/python-docs-samples:masterfrom
dwardu89:masterdwardu89/python-docs-samples:masterCopy head branch name to clipboard
Jan 17, 2017
Merged

Updated README.md#759
theacodes merged 1 commit into
GoogleCloudPlatform:masterGoogleCloudPlatform/python-docs-samples:masterfrom
dwardu89:masterdwardu89/python-docs-samples:masterCopy head branch name to clipboard

Conversation

@dwardu89

Copy link
Copy Markdown
Contributor

Updated the command gcloud beta app deploy to be gcloud app deploy, since the command is no longer in beta.

@googlebot

Copy link
Copy Markdown

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@googlebot googlebot added the cla: no This human has *not* signed the Contributor License Agreement. label Jan 15, 2017
@dwardu89

Copy link
Copy Markdown
Contributor Author

I signed it!

@googlebot

Copy link
Copy Markdown

CLAs look good, thanks!

@googlebot googlebot added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels Jan 15, 2017
@theacodes

Copy link
Copy Markdown
Contributor

I think this one is intentionally still beta because of endpoints.

@lookuptable @wlu2016 @qiwzhang?

@qiwzhang

Copy link
Copy Markdown
Contributor

The command is "gcloud app deploy" now. Please refers to this

https://cloud.google.com/endpoints/docs/quickstart-app-engine#python

Thanks for updating the README.

@theacodes theacodes merged commit ef86b6f into GoogleCloudPlatform:master Jan 17, 2017
@theacodes

Copy link
Copy Markdown
Contributor

@qiwzhang it seems the quickstart no longer even uses this particular sample (instead uses /endpoints), should we just remove this one?

@qiwzhang

Copy link
Copy Markdown
Contributor

It is better to leave it here. Maybe there are still some docs referring to here.

@theacodes

Copy link
Copy Markdown
Contributor

I can search and verify, better to not have a duplicated sample. :)

@qiwzhang

qiwzhang commented Jan 17, 2017 via email

Copy link
Copy Markdown
Contributor

chalmerlowe pushed a commit that referenced this pull request Feb 19, 2026
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
chalmerlowe pushed a commit that referenced this pull request Feb 19, 2026
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
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.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.