13

Recently, in the Rust documentation, someone suggested an edit to try and clarify an example, but it contained several grammatical errors and changed the meaning to be incorrect (traits in Rust aren't linked to interfaces, as the edit would have suggested; they are likened as the original text said). Therefore, I rejected the edit as incorrect, but the change was approved overall and added into the documentation.

What is the correct action in this case? I proposed a revert of the change, which is currently awaiting review, but I was wondering whether it would also be appropriate to flag for a moderator, considering that 3 users approved the change which was clearly not a good change. In my opinion, if a reviewer doesn't even notice that the grammar is incorrect, then they aren't reviewing very thoroughly at all (perhaps another manifestation of the robo-reviewers?)

In summary, should I flag in the case of blatantly incorrect reviews, or just resolve the issues myself?

12
  • 5
    two of the three i immediately recognize as common problem reviewers.
    – Kevin B
    Commented Sep 22, 2016 at 19:53
  • One of the users (zaq) is actually the topic of another meta thread, it turns out. I hope it's not auto-approving changes that look OK (but aren't!).
    – Aurora0001
    Commented Sep 22, 2016 at 19:55
  • 1
    He's also one of the ones that approved your fix to the incorrectly approved one.
    – Kevin B
    Commented Sep 22, 2016 at 19:56
  • 1
    @Aurora0001 He also approve this one. He really needs to be stopped. Commented Sep 22, 2016 at 20:13
  • 26
    @NathanOliver - Those reviewers won't be doing so for a while. A larger question: why does it make sense that someone with no posts at all on Stack Overflow is able to review Documentation edits? Commented Sep 22, 2016 at 20:43
  • @BradLarson that is a very good question. I think it should be a 2k privilege just like regular edits. Thanks for handling the reviewers. Commented Sep 22, 2016 at 20:55
  • @NathanOliver IMO reviewing doc edits shouldn't have a rep limit that high. After all, isn't everything in Documentation supposed to be sort of a wiki?
    – SE is dead
    Commented Sep 22, 2016 at 21:15
  • 3
    Certainly higher than 1, though @dorukayhan? I would think at least 100 (edit community wiki on QA) would help... Commented Sep 22, 2016 at 21:23
  • 5
    @MikeMcCaughan It's already set at 100. I personally would rather have it at 500, where the "access review queues" privilege is set.
    – Kendra
    Commented Sep 22, 2016 at 21:34
  • But the reviewer mentioned here only had 1 rep @Kendra? Commented Sep 22, 2016 at 21:43
  • 1
    @MikeMcCaughan User is currently suspended, they actually had 130+ when I checked a couple days ago. Edit: 138 rep, check their profile and their rep tab. Association and edits
    – Kendra
    Commented Sep 22, 2016 at 21:44
  • Oh, I totally forgot about suspension making the rep 1... Nevermind :) Commented Sep 22, 2016 at 22:04

0

Browse other questions tagged .