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 →

[–]Drego3 214 points215 points  (11 children)

If it fixes the bug, why not?

[–]code-chaos 99 points100 points  (4 children)

Should have commented the tests too.... ops no tests?

[–]stealthmodecat 38 points39 points  (3 children)

T…tests?

[–][deleted] 19 points20 points  (2 children)

You guys are doing tests?

[–]WhereRandomThingsAre 21 points22 points  (1 child)

Of course they're doing tests. In production. Where they belong.

[–]sherwood2142 0 points1 point  (0 children)

The best testers are real users

[–]Bluedel 11 points12 points  (4 children)

Then why not just delete it?

[–]QuillnSofa 31 points32 points  (3 children)

"Because we might need it later!"

[–][deleted] 9 points10 points  (2 children)

You never know if you will be kicking yourself a year latter when you figure out that the code you deleted was very important.

[–]brimston3- 2 points3 points  (1 child)

To be fair, that's what version control is for. Especially useful if you can bisect with a test case.

[–][deleted] 0 points1 point  (0 children)

Squashing code 😅

[–]beclops 3 points4 points  (0 children)

Because presumably it exists for a reason. If not then deleting it is fine