Skip to content
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

Update development mode so that changes to static/js files are reflected without needing a process restart #1661

Open
Zequez opened this issue Sep 8, 2021 · 3 comments

Comments

@Zequez
Copy link
Contributor

Zequez commented Sep 8, 2021

Hi! At the moment of writing, whenever you change one of the static/js files the changes are not copied to the /dist folder. I'm not sure if this is intentional, but it makes for a poor development experience, as the development process takes about 30 seconds to restart.

Anyway, right now I changed the copy-to-dist.mjs file and added a --watch option and a watch:static script to package.json and I've got myself the functionality I needed locally. Would there be any interest into making this a formal PR? Also, what's the deal with the duplicated scripts on build-tools? I found no references to the.js versions of the scripts that are also on the .mjs format, could they be safely removed?

On a personal note I would advocate for using something other than Webpack, like Snowpack or Vite and take advantage of ESM module loading on the browsers.

@vfat0
Copy link
Member

vfat0 commented Sep 9, 2021

Hiya! All this stuff has just been copied over from the original (yieldfarming.info), I still don't know what an mjs file is a year later :) A PR would be very much welcome.

@G-Lex
Copy link

G-Lex commented Oct 12, 2021

@Zequez Did you get around to making the changes for the PR?

@imasters11
Copy link

@Zequez I would appreciate this as well

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

No branches or pull requests

4 participants