search

firefox-devtools / rfcs

RFCs for changes to DevTools
15 stars 17 forks source link

Remove loader hotreload feature #18

Closed juliandescottes closed 7 years ago

juliandescottes commented 7 years ago

(adding comment from @bgrins from the email thread)

I’m in favor of removing it. By the way, I’m hoping to expose similar functionality from ./mach watch in https://bugzilla.mozilla.org/show_bug.cgi?id=1388899, which will work cross platform and for the entire frontend (including devtools).

juliandescottes commented 7 years ago

(adding comment from @captainbrosset from the email thread)

Sounds good to me. Removing code always sounds good.

juliandescottes commented 7 years ago

(adding comment from @sole from the email thread)

I like the idea of having a similar functionality cross platform, so this is a +1 from me

Question: didn't we want to run this kind of proposals through the process here? https://github.com/devtools-html/rfcs/

Or is it not substantial enough?

Thanks

Also YAY DELETING!

sole

juliandescottes commented 7 years ago

(adding comment from @ZER0 from the email thread)

+1 for removing; especially if we're going to have a similar functionality cross platform.

juliandescottes commented 7 years ago

Closing the PR in preparation for repository cleanup.