Timeline for When reviewing edits, should I reject edits that introduce nontrivial "fixes"?
Current License: CC BY-SA 3.0
24 events
when toggle format | what | by | license | comment | |
---|---|---|---|---|---|
May 23, 2017 at 12:38 | history | edited | CommunityBot |
replaced http://stackoverflow.com/ with https://stackoverflow.com/
|
|
Aug 22, 2016 at 19:09 | history | edited | Peter Mortensen | CC BY-SA 3.0 |
added 4 characters in body; edited title
|
Aug 22, 2016 at 10:19 | comment | added | Tasos Papastylianou | @LucasTrzesniewski I had not even thought of that. well spotted. | |
Aug 22, 2016 at 9:35 | comment | added | Gimby | Just to note: I dropped "automatically" from the title. Maybe I'm the only one, but when seeing the title in the list of questions my "robo reviewing" alarm went off and had to be quieted by the otherwise perfectly sensible body of your question :) If you don't agree, feel free to revert. | |
Aug 22, 2016 at 9:33 | history | edited | Gimby | CC BY-SA 3.0 |
remove "automatically" from the title, hints too much towards robo-reviewing which is not relevant at all
|
Aug 22, 2016 at 8:57 | comment | added | Lucas Trzesniewski | Note that the user which suggested the edit does not have enough rep to comment. | |
Aug 22, 2016 at 7:46 | history | edited | Tasos Papastylianou | CC BY-SA 3.0 |
edited title
|
Aug 22, 2016 at 7:12 | answer | added | Steve Bennett | timeline score: 6 | |
Aug 21, 2016 at 18:54 | vote | accept | Tasos Papastylianou | ||
Aug 21, 2016 at 18:52 | answer | added | user4639281 | timeline score: 15 | |
Aug 21, 2016 at 18:43 | comment | added | Tasos Papastylianou | @TinyGiant thank you for the clarification below. If you'd like to post that as an answer I will accept it. | |
Aug 21, 2016 at 18:38 | comment | added | user4639281 | I don't see any hostility here, nor is anyone accusing you of trolling. | |
Aug 21, 2016 at 18:36 | comment | added | Tasos Papastylianou | @TinyGiant I don't understand where all this hostility is coming from. My question is very similar to meta.stackoverflow.com/questions/267845/… except from the reviewer's point of view. In the words of that post, should "introduction of code that changes functionality, even if the editor thinks it's correct" be rejected on that basis? I don't see why I'm being viewed as trolling. I want my reviewing process to be in line with the guidelines of the community, and in this particular case it's unclear and I'm asking for clarification. | |
Aug 21, 2016 at 18:22 | comment | added | user4639281 | As I said below, the edit is not an attempt to reply. It is not a comment being edited into a post. Whether the edit is correct or not is a different story. But in reality the question is the problem, it is specifically asking for opinions on the topic. Everything else is symptoms of the problem | |
Aug 21, 2016 at 18:19 | comment | added | Tasos Papastylianou | @TinyGiant specifically it's "discussion" based. I'm trying to ascertain to what extent the "reject due to attempt to reply-style edits" clause applies to answer posts as well as question posts (where the case is clear). | |
Aug 21, 2016 at 18:18 | comment | added | Tasos Papastylianou | apologies for my brainfart, I didn't mean to say "accept" I meant to say "reject" | |
Aug 21, 2016 at 18:18 | history | edited | Tasos Papastylianou | CC BY-SA 3.0 |
corrected my brainfart. Now I realise why people were saying "you shouldn't accept anyway" I never meant to say accept :p
|
Aug 21, 2016 at 18:17 | comment | added | user4639281 | close vote that question, it is blatantly opinion based | |
Aug 21, 2016 at 18:05 | comment | added | Floern | Considering the question to that answer, I think it's not even relevant whether the give code runs correctly, since it's just an arbitrary example for code style. The only thing this edit does, is blowing up the example code and messing up the code indentation.. | |
Aug 21, 2016 at 18:04 | history | edited | Shog9 |
edited tags
|
|
Aug 21, 2016 at 18:03 | answer | added | Shog9 | timeline score: 17 | |
Aug 21, 2016 at 17:58 | comment | added | Tasos Papastylianou | that's a fair point, but it's clear to me this is not a trivial fix, so. the real question is should I have straight up rejected for that reason, or just skip. | |
Aug 21, 2016 at 17:54 | comment | added | Cody Gray Mod | Skipping it was the right move. Putting aside the philosophical debate about whether these types of edits are recommended, as someone who is (according to your profile) not a Java developer, you would not be qualified to judge the veracity of such an edit. Me neither. But of our countless members, someone knows for sure whether or not this is a good edit. | |
Aug 21, 2016 at 17:51 | history | asked | Tasos Papastylianou | CC BY-SA 3.0 |