I've never felt like our variable names in v1 were too short in my opinion.

I've had this bite me a few times, I think it's too sensitive. I agree with it in theory but it wouldn't let me use a single character variable for a 4 line middleware function. Idiomatic Go would say that is totally fine as long as it's not a super long function and doesn't leave scope.

I generally don't love using # of lines as the metric for this.

I've personally found this to be quite helpful in producing readable code. I think that edge-case of specific types matched with specific names can be helpful to declare.

It's an arbitrary position, but I'd rather have consistency withrw http.ResponseWriter over a mix ofw andrw than the shorter name.

Didn't realize my request for change would block merge. Since everyone on the team seems to agree with this, I shall approve too.