--reload_dirs option does not behave as documented #1783
Unanswered
ImogenBits
asked this question in
Potential Issue
Replies: 2 comments
-
It would be really nice to remove the piece of code @ImogenBits mentioned as it leads to log spam when the logging goes to a file in the working directory. |
Beta Was this translation helpful? Give feedback.
0 replies
-
I only noticed this because it's causing me to run out of inotify handles. I guess I need to chdir into the directory rather than relying on the --app-dir and --reload-dir options, which do not work as advertised. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
The docs describe the option like this (emphathis mine):
But this is not how it actually behaves with when using the watchfile optional dependency. Currently it instead always watches the current directory in addition to whatever is specified by
reload_dirs
.The cli output also is wrong since it only lists the directories passed to
reload_dirs
.This seems to be caused by this bit of code. Since it adds the cwd into the watched directories after the config setup it also won't actually appear anywhere visible to the end user such as the cli output.
Beta Was this translation helpful? Give feedback.
All reactions