36

In this question: Javascript Reading only first value the OP wrote what the expected result is in the comments.

I made an edit to include the comment so people won't need to go over all the comments to find it.

Although my edit note says "Added expected result (as noted in the comments)" the reviewers rejected it as "This edit was intended to address the author of the post and makes no sense as an edit. It should have been written as a comment or an answer."

They obviously didn't read the comment section... Usually I wouldn't expect a reviewer to read the comment section, but in this case I specifically referred to it.

  1. Do you think the edit is useful?
  2. What can I do to make the intent of the edit clearer for reviewers?
  3. Assuming not all reviewers read the edit comment, is it acceptable to write the comment inside the edit itself?
2
  • 20
    "is it acceptable to write the comment inside the edit itself?" - no, not at all. Reviewers that don't read the edit message are bad reviewers and should be banned from reviewing.
    – Bergi
    Commented Jan 20, 2019 at 15:03
  • 2
    I don't think it should be done either. That's why I didn't do it. But adding Edit As mentioned in the comments, the expected... might be more effective to achieve the end goal which is to have a more complete question. So, if it is more effective, it might also be acceptable.
    – Oram
    Commented Jan 20, 2019 at 15:21

2 Answers 2

39

Editing the question to add relevant information needed to answer the question was the right move, and something OP should've done themselves. So, I'd say this was an oopsie on the reviewers' part.

Perhaps you could have made the fact that this was info given by OP obvious by including a link to the relevant comment(s) in the edit summary.

Editing to include information provided by OP in the comments. Relevant comment: https://stackoverflow.com/q/54263023#comment95350416_54263023

7
  • 14
    Verbosity never hurts, but I'd say the message already is fine without the link. "Adding expected results as given by OP in comments"
    – Bergi
    Commented Jan 20, 2019 at 15:05
  • 25
    tl;dr reviewers are idiots, get used to it :) Commented Jan 20, 2019 at 15:53
  • 10
    I honestly prefer the OP's original comment, and think speculating about how fairly meaningless tinkering with wording might've improved the edit's chances just shifts a portion of blame onto the suggester that he doesn't deserve. The link to the specific comment isn't needed here, and adds nothing; I'm pretty sure it won't even render as a clickable link in the review queue, so it doesn't even provide a more convenient way for a reviewer to get to the comment than the question title does. I don't think there's much more to be said about this case than what @LightnessRacesinOrbit has said.
    – Mark Amery
    Commented Jan 21, 2019 at 10:06
  • 2
    @MarkAmery I did say it "was an oopsie on the reviewers' part". You can't help stupidity but you can always guard yourself against it. Perhaps you have heard of the term "defensive driving".
    – cs95
    Commented Jan 22, 2019 at 17:54
  • 1
    I don't dispute that in principle, @coldspeed, but I don't think that any of the tweaks to the edit summary that you propose here do in fact help defend against the stupidity, even slightly.
    – Mark Amery
    Commented Jan 22, 2019 at 17:56
  • 1
    @LightnessRacesinOrbit well, I was going with lack of coffee meta.stackexchange.com/q/189562/213575
    – Braiam
    Commented Jan 22, 2019 at 18:24
  • @Braiam Coffee laced with cyanide perhaps ahem Commented Jan 23, 2019 at 1:54
23

Your edit was fine. You explicitly highlighted that you were moving information from the comments into the question. I've checked the comments, and I see that you were indeed faithfully transcribing the OP's writing. I'm not convinced that there's any way you could've highlighted this more clearly.

What's more, even if you hadn't made this clear in your edit summary, the rejection reason chosen by the reviewers would've made no sense. I can see how a reviewer who missed that detail might've wrongly thought the edit deviated from the original author's intent, but it's plainly not an attempt to address the author. That rejection reason is meant to be for people who try to edit stuff like "Good question, did you ever solve this?" or "But what output are you expecting?" or "Try this! foo(bar)" directly into the question, not for elaborations on the question that the reviewer thinks are misguided.

I'm gonna raise a flag suggesting that both reviewers get a message pointing to this question. The quality of suggested edit reviewing is in general so bad that trying to improve it feels hopeless, especially since those of us who care don't have the main tool that we need to do anything about it, but I guess that, in the absence of any non-aggressive remedy, I can at least unleash the attack dogs moderators and hope that something useful comes out of it.

1
  • 3
    Oh man, that feature request is truly a thing of beauty... Commented Jan 22, 2019 at 17:11

You must log in to answer this question.

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