I think only calling the `setup` functions of linters that are running is very reasonable (and even desirable), but it's also adding a little scope bloat. So unless you're motivated to make that change, it might just be best to just move that logic back to the start of the `lint` function (I'll leave this up to you). I know I originally said to move it to set up, sorry for the churn :(.
Bug 1353680 Comment 18 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
I think only calling the `setup` functions of linters that are running is very reasonable (and even desirable), but it's also adding scope bloat. So unless you're motivated to make that change, it might just be best to just move that logic back to the start of the `lint` function (I'll leave this up to you). I know I originally said to move it to set up, sorry for the churn :(.
I think only calling the `setup` functions of linters that are running is very reasonable (and even desirable), but it's also adding scope bloat. So unless you're motivated to make that change, it might just be best to just move that logic back to the start of the `lint` function (I'll leave this up to you). I know I originally said to move it to `setup`, sorry for the churn :(.