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

Make script reloading dynamically configurable #4064

Closed
clintongormley opened this issue Nov 4, 2013 · 1 comment
Closed

Make script reloading dynamically configurable #4064

clintongormley opened this issue Nov 4, 2013 · 1 comment

Comments

@clintongormley
Copy link

Re #4062

Not sure if we need two separate settings for script reloading, ie watcher.interval and script.auto_reload_enabled. I'd make it just one: script.auto_reload.interval which defaults to -1.

I'd also like it to be dynamically configurable, so that I can turn it on (with a short interval) while making changes, then turn it off again.

Perhaps we should keep script.auto_reload_enabled and default it to true, but make it non-dynamic, to avoid a potential exploit.

@ghost ghost assigned imotov Nov 4, 2013
@clintongormley clintongormley assigned GaelTadh and unassigned imotov Jul 4, 2014
@clintongormley
Copy link
Author

Meh, changed my mind. Closing

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

3 participants