On Wed, Nov 29, 2023 at 6:38 AM Marco Bonardo <[email protected]> wrote:

> In this case I prefer to add the new unused argument to all the
> interfaceImpl instances during the refactoring, so when DeveloperB arrives,
> they will find:
> => file.js
> interfaceImpl(arg1, arg2, something) {
>   return arg1 + arg2;
> }
>

How do you feel about doing this instead?
interfaceImpl(arg1, arg2 /*, something */) {
  return arg1 + arg2;
}

I'd probably add a comment explaining why I didn't need 'something', but
that's a side issue.

-Dan Veditz

-- 
You received this message because you are subscribed to the Google Groups 
"[email protected]" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/a/mozilla.org/d/msgid/dev-platform/CADYDTCBnLzsQ_YP6nQPy5BTPiOAwV8%3Drhz5%2BL0Gs4O%2BvmSP2Yg%40mail.gmail.com.

Reply via email to