9

senior: we have no reason to change/mutate this property, we don't need a setter method, just set it once in the constructor

also senior: there's no reason to change/mutate this value, but i want you to write a setter method for it

Comments
  • 4
    At least they have the memory of a senior
  • 1
    The same senior ?
    I’m with split personality number one, use the constructor.
  • 1
    cherry on top i forgot to mention

    the new setter method proposed does't even get called anywhere in my PR....

    yes same senior
Add Comment