chore: add cf pages deploy script to vue example #1562
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "scripts": { | ||
| "dev": "vite" | ||
| "dev": "vite", | ||
| "build": "vite build", | ||
| "preview": "vite preview", | ||
| "deploy": "npm run build && wrangler pages deploy dist" |
There was a problem hiding this comment.
The new deploy script relies on the wrangler CLI, but the package.json does not declare wrangler in devDependencies, so a fresh install (npm install then npm run deploy) will fail with wrangler: command not found unless the user already has a global install. This makes the added deployment path unusable out of the box; bundling wrangler locally or invoking it via npx would avoid the failure.
Useful? React with 👍 / 👎.
mattConnHarbour
force-pushed
the
SD-1201
branch
from
418e46e to
a71ce63
Compare
harbournick
force-pushed
the
main
branch
2 times, most recently
from
84f8ea5 to
2a443d8
Compare
No description provided.