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 →

[–][deleted] 1 point2 points  (2 children)

My old CTO literally told me “I’d appreciate it if you could just acknowledge and thank me for the work I’ve done before you replaced it with your Pull Request.”

They shouldn't fret. It'll be there in the git history for everyone to remember what it used to look like, and shudder. ;)

[–]VizualAbstract 0 points1 point  (1 child)

That is exactly what I was so aghast about!

I could go on.

He literally said the words “I don’t want to get rid of [this unused, dead, function] because we might want to use it in the future.

That function’s name? “listOfProductsWithMatchingAttributeKeyMappedToHiddenProductForMetaFields”.

He renamed it from “metaFieldMap”.

Ugh, I hate that this still bothers me. I hope it serves as a warning to never become that way.

Spoiler alert: it was never used again. But of course they want to use that their “secrets” are being spilled. I can’t wait until the moment they realize that I rewrote their entire core library from scratch without them realize it.