I know I've posted about this before, but it's gotten worse since then. It's become a policy to mark lines that were changed, according to their TFS/JIRA numbers.
I argued that blame tools with the source control could let us know exactly what a code change was related to, as long as commit comments contained the TFS task number, which is already a process standard we follow.
This ends up looking like this
method(int b)
{
// TFS-123123 Start
a = 5;
c = 10;
// TFS-123123 End
return a + b;
}
Apparently, it's very important that people that don't have access to our source control tools can trace back errors, because the delivery teams still look at code when trying to research an issue before they escalate to us.
But apparently that wasn't enough.
And now this is becoming popular
method(int b)
{
// TFS-123123 Start
// a = 3;
// c = 8;
// a should be 5, and c should be 10;
a = 5;
c = 10;
// TFS-123123 End
return a + b;
}
Please... make it stop.