django icon indicating copy to clipboard operation
django copied to clipboard

Refs #30213 -- Removed post-startup check for Watchman availability.

Open carltongibson opened this issue 1 year ago • 0 comments

This is checked at startup in get_reloader(). The runtime check ties the implementation to Watchman excessively.

Hey @orf 👋 — can I ask for your thoughts here? I was looking at ticket-30213, and this double-check seems a bit excessive… 

  • Is it likely Watchman is available at startup but then becomes unavailable? (get_reloader() already checked it was available... 🤔)
  • Is that worth privileging in relation to other reloader strategies?
  • Can we remove this check?

Sorry if this is re-treading old ground.

Thanks! 🎁

carltongibson avatar Jul 05 '22 08:07 carltongibson