-
Notifications
You must be signed in to change notification settings - Fork 140
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
Vite is trying to compile the scripts/parsers/in contents #722
Comments
I would like to work on this issue |
Hey @davepagurek |
I think that's fine @webermayank! |
Okay i'll do some final tests then raise a PR |
Should this be closed now that the PR is merged @davepagurek ? Or what remains to be done? |
Most appropriate sections of the p5.js website?
Other (specify if possible)
What is your operating system?
Windows
Web browser and version
N/A
Actual Behavior
When running
npm run dev
, sometimes we run into Vite errors because it's trying to compile the js code inscripts/parsers/in
and you get an error message from the internals of p5.sound. @lukeplowden and I have also found that the time from starting the dev server to being able to load the site is noticeably faster (at least on windows) after deleting the contents of that folder.Expected Behavior
The
scripts/parsers/in
folder is mostly just used to produce the reference material. So far, we keep it around based on the assumption that cloning and setting up the mainp5.js
would be slower if we didn't, but we don't update the website reference all that often (just when a new version is released) compared to how often we make other changes to the site. So maybe it would be best if the reference parser script clears out thescripts/parsers/in
folder after it's done? (Possiblyscripts/parsers/out
too?)Steps to reproduce
npm run build:reference
npm run dev
Would you like to work on the issue?
Someone else can take this on if desired!
The text was updated successfully, but these errors were encountered: