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

Add statreload config option reload_types #550

Closed
wants to merge 1 commit into from

Conversation

WillDaSilva
Copy link

Solves #528
Helps #102

reload_types is a config option that allows the specification of multiple file suffixes to be monitored.

reload_types is a config option that allows the specification of
multiple file suffixes to be monitored.
@rafalp
Copy link
Member

rafalp commented Apr 8, 2020

Not sure if I like an additional option here. We could simply watch everything that is not:

  • .dotted: because it's dev tools configs and/or caches
  • .pyc: because changes in those are triggered by changes in .py

We've also merged an alternative strategy for watching changes that uses file system events (aka the watchdog strategy) that will also need updating.

@rafalp rafalp requested review from a team and removed request for tomchristie April 8, 2020 13:46
@euri10 euri10 mentioned this pull request Apr 11, 2020
2 tasks
@rafalp
Copy link
Member

rafalp commented Apr 23, 2020

Thank you for taking the time to open PR! While it wasn't merged, it was helpful as a reference for me <3

@rafalp rafalp closed this Apr 23, 2020
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

Successfully merging this pull request may close these issues.

2 participants