4

This original post included a Django solution prior to Django version 1.6:

import warnings
warnings.filterwarnings(
    'error', r"DateTimeField received a naive datetime",
    RuntimeWarning, r'django\.db\.models\.fields')

I suggested to edit this to make it work with Django > 1.6:

import warnings
warnings.filterwarnings(
        'error', r"DateTimeField .* received a naive datetime",
        RuntimeWarning, r'django\.db\.models\.fields')

and included a comment to this effect. The edit was rejected on grounds of "deviating form the original intent". Needless to say, the OP made the very same change just a couple of weeks later.

Why should I invest time in suggesting useful edits if they get rejected, with all due respect, by people who quite obviously don't understand the subject matter?

Is there an appeal system to rejected edits?

Edit: feature request

Following up on below comments, I'd like to suggest that the review rights get revised.

Instead of allowing reviews by reputation only OPs, previous editors and authors of accepted or positively voted answers on the same question as the edit-suggested entry get first-reviewer rights. If the review doesn't get voted on in some fixed time, the review rights could be extended to people who have a demonstrated knowledge of the topic, e.g. by matching up tags and previous comments.

That is the final vote on whether a suggested edit gets accepted or rejected should be by those who are actively involved in a thread, rather than just about anyone. This way edits actually get peer reviewed.

16
  • edits in code are frowned upon, they are easier accepted on questions as long as they don't fix the problem of the OP. On answers code edits are better left to the original OP by leaving a comment.
    – rene
    Commented Apr 12, 2015 at 14:36
  • 7
    @rene IMHO that's just broken. So I have to tell the OP "hey, this is not working anymore, please fix it (I'd love to show you, but edits in code get rejected anyway, so I'll leave it up to you to invest the same amount of time that I just did, have fun!)? Quite frankly, that just shows how broken SO's review system is.
    – miraculixx
    Commented Apr 12, 2015 at 14:38
  • It's a slippery slope because that might just have been a bad edit by someone who doesn't know what he's doing. Ideally edits to code should be reviewed by people who are well versed in that language but that comes with many problems of its own. The decision to forbid it altogether is the lesser of two evils. Commented Apr 12, 2015 at 14:44
  • 1
    The system is fine, the users are at fault here. if they would skip things they can't judge things would go smoother. Until every user realized that we have to accept that our good intentions sometimes gets blocked by robo-reviewers.
    – rene
    Commented Apr 12, 2015 at 14:44
  • @JeroenVannevel well, isn't the peer review system supposed to guarantee that edits make sense? If it can't guarantee that but rather favors rejection over acceptance for any "complicated" edits (which this one clearly isn't, it was well explained by the additional comment), it is just a bureaucratic system, and as such doesn't deserve the label 'peer review'.
    – miraculixx
    Commented Apr 12, 2015 at 14:50
  • downvoters, please explain. That's another bad habit on SO and here.
    – miraculixx
    Commented Apr 12, 2015 at 14:51
  • 1
    I quit the review queues a year ago for a reason: it's filled with people that simply approve everything. Placing faith in the reviewers is a gamble which often does not work out. Commented Apr 12, 2015 at 14:52
  • 1
    @miraculixx you have tagged this post as feature request. Downvotes on those kind of posts indicate no, thank you don't implement that FR and maybe someone lost his keys...
    – rene
    Commented Apr 12, 2015 at 15:00
  • 2
    Tip: Don't tack a feature-request on the end. Make it its own clean post, maybe linking this as motivation. Commented Apr 12, 2015 at 15:05
  • Obligatory read Is it OK to edit the question to change the author's intention
    – Braiam
    Commented Apr 12, 2015 at 15:46
  • The odds that you'll get a reviewer that knows anything about django versions is just too low. They'll choose the safe way. Commented Apr 12, 2015 at 16:47
  • @HansPassant safe for whom? that's why I'm proposing to change the way reviews work.
    – miraculixx
    Commented Apr 12, 2015 at 16:51
  • Safe for future readers of course. I'll propose you earn another 1800 rep so you don't have to be reviewed anymore and can make these kind of edits at your heart's content. Commented Apr 12, 2015 at 16:57
  • @HansPassant Well that's my whole point, the reviewers' decision was not safe for future readers... On your suggestion to earn more rep, I'll take that with a grain of salt - however, if I was only looking out for myself I wouldn't be on SO in the first place ;-)
    – miraculixx
    Commented Apr 12, 2015 at 17:00
  • Why not just add an answer with the updated code? Link back to the original answer to make clear where the original code came from, and add text around what you changed to make it work with the new version. If you feel bad about getting rep for someone else's answer, make it a community wiki. That seems like an even safer bet than editing other people's answers... Commented Apr 13, 2015 at 14:20

2 Answers 2

8

You are right, and the reviewers were wrong.

There's this notion on Stack Overflow that editing code is a taboo. That's wrong.

As long as the intent wasn't changed, code editing is just fine. Don't let it discourage you.

9
  • Though it is risky, as edits to code are harder to judge. Twice that in questions. And naturally, it must still work for the original version... Commented Apr 12, 2015 at 15:03
  • it must still work for the original version... well, it does.
    – miraculixx
    Commented Apr 12, 2015 at 15:08
  • 2
    @Deduplicator if it's too hard to judge, don't judge. It's that simple. Rejecting mindlessly because "o noez it haz codez in ze edit!" is not a good move. Commented Apr 12, 2015 at 15:29
  • @Braiam I didn't edit the question.
    – miraculixx
    Commented Apr 12, 2015 at 15:58
  • @miraculixx is the same guidance for all edits.
    – Braiam
    Commented Apr 12, 2015 at 15:59
  • @Braiam my edit didn't change the author's intention either.
    – miraculixx
    Commented Apr 12, 2015 at 16:01
  • @miraculixx if you read the answer, it says that "intention" doesn't matter: Crucially, there's nothing about intent there; you won't always know what the intentions of the author were until he clarifies by commenting, accepting or reverting your edit.
    – Braiam
    Commented Apr 12, 2015 at 16:04
  • @SecondRikudo: Yes. Which doesn't change that many judge even if they aren't qualified, though that's more often the accept-bots. Commented Apr 12, 2015 at 17:31
5

Lets see what the help center says:

When should I edit posts?

Any time you see a post that needs improvement and are inclined to suggest an edit, you are welcome to do so. The original author of a question or answer may always edit their own post, regardless of reputation level.

Edits are expected to be substantial and to leave the post better than you found it. Common reasons for edits include:

  • To fix grammar and spelling mistakes
  • To clarify the meaning of the post (without changing that meaning)
  • To include additional information only found in comments, so all of the information relevant to the post is contained in one place
  • To correct minor mistakes or add updates as the post ages
  • To add related resources or hyperlinks

So, if your edit makes it works with later versions of whatever you are doing, please do so. If you see a suggested edit that does, please approve it. If you are unsure, skip it.

You must log in to answer this question.

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