Skip to content

fix(create-cloudflare): use wrangler deploy instead of wrangler publish#3486

Merged
petebacondarwin merged 3 commits intocloudflare:mainfrom
Cherry:fix/c3-deploy-scripts
Jun 21, 2023
Merged

fix(create-cloudflare): use wrangler deploy instead of wrangler publish#3486
petebacondarwin merged 3 commits intocloudflare:mainfrom
Cherry:fix/c3-deploy-scripts

Conversation

@Cherry
Copy link
Copy Markdown
Contributor

@Cherry Cherry commented Jun 17, 2023

Fixes # [insert GH or internal issue number(s)].

What this PR solves / how to test:

This fixes C3 scripts and READMEs to use the wrangler 3 deploy command instead of publish.

Author has included the following, where applicable:

Reviewer is to perform the following, as applicable:

  • Checked for inclusion of relevant tests
  • Checked for inclusion of a relevant changeset
  • Checked for creation of associated docs updates
  • Manually pulled down the changes and spot-tested

@Cherry Cherry requested review from a team, RamIdeas, jculvey and petebacondarwin as code owners June 17, 2023 16:32
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Jun 17, 2023

🦋 Changeset detected

Latest commit: 6ed8af7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
create-cloudflare Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Cherry Cherry changed the title fix(create-cloudflare): use wrangler deploy instead of wrangler publish now fix(create-cloudflare): use wrangler deploy instead of wrangler publish Jun 17, 2023
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 17, 2023

A wrangler prerelease is available for testing. You can install this latest build in your project with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/5330637794/npm-package-wrangler-3486

You can reference the automatically updated head of this PR with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/3486/npm-package-wrangler-3486

Or you can use npx with this latest build directly:

npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/5330637794/npm-package-wrangler-3486 dev path/to/script.js
Additional artifacts:
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/5330637794/npm-package-cloudflare-pages-shared-3486

Note that these links will no longer work once the GitHub Actions artifact expires.

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 17, 2023

Codecov Report

Merging #3486 (6ed8af7) into main (528cc0f) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3486      +/-   ##
==========================================
+ Coverage   75.08%   75.10%   +0.02%     
==========================================
  Files         183      183              
  Lines       11084    11084              
  Branches     2918     2918              
==========================================
+ Hits         8322     8325       +3     
+ Misses       2762     2759       -3     

see 2 files with indirect coverage changes

@Cherry Cherry force-pushed the fix/c3-deploy-scripts branch from 2bef5c6 to 1ae0fc6 Compare June 19, 2023 15:21
@petebacondarwin petebacondarwin self-assigned this Jun 21, 2023
@petebacondarwin petebacondarwin force-pushed the fix/c3-deploy-scripts branch from 1ae0fc6 to 6ed8af7 Compare June 21, 2023 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants