-
Notifications
You must be signed in to change notification settings - Fork 1.3k
refactor(adapters): modify build scripts to include additional vite b… #7504
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
Conversation
…uild step The build scripts in all adapters' package.json files were updated to include an additional `vite build` step before the existing build command. This ensures a more robust build process by performing a standard build before the adapter-specific build.
🦋 Changeset detectedLatest commit: 0866fc4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
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 |
commit: |
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
The problem is that when you run By doing the client build twice you're making things slower for everyone. So I cannot accept this PR, but if it is really necessary to make sure client is up to date, perhaps a command can be added to For example |
@wmertens |
…uild step
The build scripts in all adapters' package.json files were updated to include an additional
vite build
step before the existing build command. This ensures a more robust build process by performing a standard build before the adapter-specific build.What is it?
fixes #7479
Description
Checklist
pnpm change