Bug 1733813 Comment 5 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Hi Sebo, yes, sorry for the delay. I wrote an eslint rule and its associated tests for this right after taking it, but unfortunately didn't get to finishing the actual replacement (since it seemed like one needed to be careful to check for places where the other behaviour was required, as in comment #2, it wouldn't surprise me if that's used in devtools code somewhere) and it fell to the wayside.  Looking back it would have made more sense to just land it with exclusions for the existing uses and then go back and remove them.
I have the patches still and will revive them, but I don't think I'll be able to properly get to it until next week.
Hi Sebo, yes, sorry for the delay. I wrote an eslint rule and its associated tests for this right after taking it, but unfortunately didn't get to finishing the actual replacement (since it seemed like one needed to be careful to check for places where the other behaviour was required, as in comment #1, it wouldn't surprise me if that's used in devtools code somewhere) and it fell to the wayside.  Looking back it would have made more sense to just land it with exclusions for the existing uses and then go back and remove them.
I have the patches still and will revive them, but I don't think I'll be able to properly get to it until next week.

Back to Bug 1733813 Comment 5