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

Support rollup's watch mode #5

Open
quotquot opened this issue Dec 12, 2017 · 6 comments · May be fixed by #30
Open

Support rollup's watch mode #5

quotquot opened this issue Dec 12, 2017 · 6 comments · May be fixed by #30

Comments

@quotquot
Copy link

When running rollup in watch mode, if a change is made to a copied asset, the modified asset should be copied again to the target directory. Right now one has to kill the watch process and re-run it for the copy to be done. It breaks the "mental flow" of development since one has to constantly remember that changes made to the edited buffer are not propagated automatically, unlike other sources. How many times have I reloaded the web page, wondered why it's not working, before realizing I forgot to kill the watch process and rerun it?

@thecristen
Copy link

A workaround is to make a change to one of the files that is watched, which will trigger the copy without having to kill the watch process.

@gerwinbrunner
Copy link

Any update on this?

@MickL
Copy link

MickL commented Feb 9, 2021

I was not sure if it is supposed to watch the files or not because there is this copyOnce option.

@mindplay-dk
Copy link

To others desperately searching for a solution to this, there's a fork here that worked for me:

https://www.npmjs.com/package/@guanghechen/rollup-plugin-copy

There is an open PR #42 from the author as well:

A comment on that PR says to use watch: { exclude: 'build/**' } - I didn't need to do that, it just worked.

@ViRuSTriNiTy
Copy link

@mindplay-dk That exclude configuration is only needed when you try to watch files in the build folder (often named dist). As I already commented in the PR, watching files in the build output doesn't make sense anyway so this setting is not needed at all. Sorry for the confusion.

@BreizhReloaded
Copy link

Any update about the PR @vladshcherbin?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
8 participants