16

As somebody who regularly goes through review queues on the main Stack Overflow site, the introduction of reviewing Documentation edits has brought upon a conundrum for me.

What are the expectations of reviews when going through this review queue?

Originally, I went through the proposed changes in the specific topics that I am knowledgeable in, seeing which changes had correct code, first of all, and all the other requirements. However, this review queue has brought upon the question - what are we really meant to be reviewing. Obviously, if I get a proposed change about Java (for example) I am going to be reading the descriptions and the prose, rather than the code. However, is there a standard that has been set in stone? If not, could we lay a plan out for that?

1

1 Answer 1

12

My chief opinion is that one should review the content to see if the example(s) are well written and fit in with the topic. That is to say, you should review for things like:

  • Correctness
  • Completeness
  • Plagiarism (make sure this isn't a thing)
  • Spelling/Grammar
  • Topicality (is a part of the topic)

I wouldn't say that there should be a set-in-stone standard, as we all have different review mentalities in general. However, the above bullet points as a rule of thumb should get you past some initial inertia.

Although to be fair, if you're still unclear as to how you can effectively review, it's better to not review anything than to make bad reviews. Sit it out until you feel the role is clearer to you.

3
  • Indeed, we don't want any rubbish review decisions! Sep 12, 2016 at 7:16
  • 18
    I would put 3 to the top. Plagiarism is real.
    – Braiam
    Sep 12, 2016 at 17:06
  • 5
    Completeness is good, but not a requirement. That's what improvement requests are for.
    – bwoebi
    Sep 12, 2016 at 18:01

Not the answer you're looking for? Browse other questions tagged .