13

The edit https://stackoverflow.com/revisions/68075911/2 was approved by a majority of 2 against 1 rejecting review. None of the reviewers was the post author.

The reject reason, as specified by the rejecting reviewer, was:

The edit does not improve the quality of the post. Changes to the content are unnecessary or make the post more confusing.

I agree with that. The previous revision better matched the code in the question and has the same effect (for this code). The edit explanation ("allow line was in an incorrect location") is not correct if I understand correctly that it claims the previous revision did not have the desired outcome.

I don't understand why the other reviewers approved it. I'm hesitant to roll it back myself, because maybe they had a reason. Should I?

2
  • 5
    It was just rejected by Ryan M♦ one minute ago, so a diamond mod agreed the edit was wrong.
    – cocomac
    Commented May 27, 2022 at 0:06
  • 14
    Just a quick FYI, you don't need to create a meta post for each and every bad review you encounter: we have BSOR chat room. It is frequented by moderators and active reviewers who can both help get the incorrect outcome overridden. Commented May 27, 2022 at 5:08

1 Answer 1

24

It should not have been approved. I've reversed the approval.


I don't know Rust, but regardless of whether the edit is correct, it definitely completely changes the answer. The question was about how to use the allow directive. Changing how it's used makes it a different answer. The editor should have posted it as their own answer, and the reviewers should have rejected it as "Clearly conflicts with author's intent".

You must log in to answer this question.

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