Change wrangler command from 'publish' to 'deploy' (#22392)#22804
Change wrangler command from 'publish' to 'deploy' (#22392)#22804ti-chi-bot wants to merge 1 commit intopingcap:release-8.5from
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Code Review
This pull request updates the Cloudflare Workers deployment command from npx wrangler publish to npx wrangler deploy in the documentation. The review feedback suggests ensuring terminology consistency by also updating related headings and descriptions that still use the term 'publish'.
|
|
||
| ``` | ||
| npx wrangler publish | ||
| npx wrangler deploy |
There was a problem hiding this comment.
To maintain consistency with the command change from publish to deploy, please also update the heading for Step 5 (line 100) and the description in the Wrangler section (line 29) to use "deploy" instead of "publish".
References
- Use consistent terminology throughout the documentation. Since the command has changed from 'publish' to 'deploy', related headings and descriptions should also be updated. (link)
This is an automated cherry-pick of #22392
First-time contributors' checklist
What is changed, added or deleted? (Required)
https://developers.cloudflare.com/workers/wrangler/deprecations/#publish
wrangler publishhas removed , and it should change towrangler deployWhich TiDB version(s) do your changes apply to? (Required)
Tips for choosing the affected version(s):
By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.
For details, see tips for choosing the affected versions.
What is the related PR or file link(s)?
Do your changes match any of the following descriptions?