I'm angry today. The customer's code reviewer blocked my delivery because variables of type "Context" are abbreviated as "ctx".

Err, ctx, cia, frm, dlg, dea, btn, txt, nsa, msg... are all industry standard variable names that are well understood by any coder in the world. They are used in favor of their full versions so often, that not using them is the surprising anomaly.

Conclusion: the reviewer isn't a dev or you ignored some ultra-strict "no abbreviations" guideline which was put in place for whatever reason.

Wrong tag. :(

While not critical and not a reason to reject a review, it‘s still a valid point: Don‘t use abbreviations!

@Oktokolo bullshit

@Lensflare What about when the reviewer's code is full of abbreviations?

"I can, you can't."

This is my case.

@Oktokolo Perfect. And there are no such restrictions, the whole code is full of abbreviations. It's really a case of vanity.

@codingfreedom Then the reviewer is a hypocrite, obviously.