4
\$\begingroup\$

This question already has an answer here:

This answer had 2 suggested edits that only golf the code further, and the edits were both approved. I personally disagree with edits that only golf the code further. Here's why:

  • It shouldn't be up to the reviewer to make sure a code edit works. That should be the job of the answerer. If an edit is accepted when it actually does not work, the answer may get unfair downvotes.
  • Usually golfing suggestions from comments come in the form of replace <a few characters of the code> with <shorter replacement> which might be easier to understand than the entire code being replaced.
  • The answerer may not want the edit to be in their answer or may want to change the edit before incorporating it into their answer. Ultimately it's up to the answerer whether they want to use someone's changes; they should not be forced to do so by an accepted edit.

I think the policy on PPCG should be changed to reject all edits that only shorten the code, and to ask people to comment the changes instead.

\$\endgroup\$

marked as duplicate by Martin Ender Sep 11 '15 at 21:37

This question has been asked before and already has an answer. If those answers do not fully address your question, please ask a new question.

  • 2
    \$\begingroup\$ Yes, that is already the recommended policy, but unfortunately not every reviewer knows about it. I've informed one of the reviewers now. Also note that the first edit was approved by the author of the answer. \$\endgroup\$ – Martin Ender Sep 11 '15 at 21:37
  • \$\begingroup\$ Where can this policy be found? All I can find are questions like this one, nothing I turn up in the Help Center or Tour Page reveals this policy. \$\endgroup\$ – Dan Henderson Oct 5 '15 at 18:36

Browse other questions tagged .