-
Notifications
You must be signed in to change notification settings - Fork 30
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
Auto-install issues #247
Comments
Hi, thanks for reporting! It's likely that this problem doesn't have a good solution, other than to end the process and ask the user to start it again. Node.js doesn't provide a way to clear the cache for ESM modules (as it did for commonjs), so after installing a package we might need to throw an error mentioning "this package was installed, please restart the server". |
After initial scaffolding (without non-Vue frameworks), activating one of those non-Vue frameworks is a one-off activity for the user, I feel the auto-install feature, if it requires maintenance against these sort of issues, might not be worth it. If the error can't be avoided, then instead of adding an additional message to restart the server, IMHO, it might be better to add documentation to add the relevant vite plugin. A nice alternative is to consider the astro way, for eg, |
iles
andvite
.Description 📖
After scaffolding a basic iles app without non-vue frameworks, and later when adding them via iles config, auto-install throws some errors.
Reproduction 🐞
npm create iles@next
(don't choose any non-vue frameworks in the prompts)npm install
iles.config.ts
. Set one of the non-vue frameworks to true, for eg,solid: true
The below error is thrown.
Run
npx iles info
andpnpm list
(ornpm list
) and provide the output:Logs 📜
If not providing a reproduction:
Output
Run
DEBUG=iles:* npm run dev
orDEBUG=iles:* npm run build
and provide the output:The text was updated successfully, but these errors were encountered: