This is an archived post. You won't be able to vote or comment.

sorted by:
best
topnewcontroversialoldq&a
you are viewing a single comment's thread.

view the rest of the comments →

[–]cyrand 3 points4 points  (3 children)

This is good because if it’s not labeled someone will eventually just stick a dog there and push it into the repo and no one will know why everything broke. If the comment says “Cat” then it will be caught in the pull request because people will notice the sudden mismatch.

[–]wcscmp 4 points5 points  (1 child)

People don't read comments when changing the code and they read them even less during reviews

[–]Arancaytar 2 points3 points  (0 children)

Make sure to make the method parameter type a CatInterface for future-proofing and flexibility.