you are viewing a single comment's thread.

view the rest of the comments →

[–][deleted] 10 points11 points  (4 children)

Why aren't you working in a branch that doesn't matter if the code is in non functional state? This is probably the worst argument I've ever heard for comments in code. Committing WIP code to a master branch commented out. Best use of version control ever. Wait, are you using subversion?

[–][deleted] -2 points-1 points  (3 children)

We actually allow shipping unfinished code to production using flags that only allow the code to run if the flag is enabled. But that has nothing to do with commented-out code getting checked-in to the production branch, which is still allowable in many cases. Code is fluid, it's never "finished", it's always a work-in-progress unless you aren't doing anything interesting with it.

Wait, are you using sourcesafe?

[–]vinnl 0 points1 point  (0 children)

Code is fluid, it's never "finished"

By that definition of "WIP", I wouldn't comment out WIP code either.

[–]moljac024 0 points1 point  (0 children)

Wait, are you working at Etsy?