-
-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
/public is added to watchlist #2602
Comments
Hi @bep Are there any files I should look at to fix this? |
@Imhotup I will have to think about this ... this may be as designed, but still is unfortunate. We are listening for changes in the site root for config.toml, and the watches seems to be recursive by default. |
OK, I see what's happening now. I was doing some work in a theme, so I have myTheme/exampleSite and set themesdir = "../.." In the exampleSite to test it. This works fine, but he/she who made the watcher dir logic for themes was lazy (probably me, then :-)) and added the entire theme dir to watch list, while doing the proper thing for the base project (adding static, content etc.) While this works, it is annoying when you have stuff like /assets/javascript etc. that you don't want to trigger reloads. |
The same behavior as for the root project. Fixes gohugoio#2602
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Noticed this when having
hugo server
and doing ahugo
on the side:That doesn't sound right.
The text was updated successfully, but these errors were encountered: