#27685: Allow autoreloader to use watchman
--------------------------------------+------------------------------------
Reporter: Aymeric Augustin | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: Utilities | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
--------------------------------------+------------------------------------
Comment (by Aymeric Augustin):
Tom: thanks for picking up the flag! I'm super happy to see this move
forwards.
I read the description of your PR; it makes a lot of sense. Having a base
class and several implementation (possibly depending on additional
software) is an excellent starting point. It would be good to merge
regardless of what comes next. I'll try to take a closer look in the
coming days. Ping me if you haven't heard of me by mid-August.
My three questions are:
- is the directory / glob pattern configuration method sufficiently
generic to work with all backends we envision?
- how does the signal-based design work? (I'm usually not a fan of
signals, but surprisingly, I have rather positive feelings about this use
case. I just want to make sure that's really the best solution.)
- how much backwards incompatibility does this patch incur? (Larger
backwards incompatibility need larger improvements to justify them.)
PS - In addition to the mailing list threads I linked above, I think the
best summary I wrote is on the
[https://code.djangoproject.com/wiki/SummerOfCode2017#Improvedautoreloader27685
GSOC Ideas page].
--
Ticket URL: <https://code.djangoproject.com/ticket/27685#comment:6>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/067.c669e98c0a17940b7cbb79d90a6d3057%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
