Timeline for How should code reviews be carried out? [duplicate]
Current License: CC BY-SA 4.0
22 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Jun 3, 2018 at 0:33 | history | edited | Graviton | CC BY-SA 4.0 | edited title |
| Apr 12, 2017 at 7:31 | history | edited | CommunityBot | replaced http://programmers.stackexchange.com/ with https://softwareengineering.stackexchange.com/ | |
| May 25, 2013 at 15:50 | history | closed | gnat BЈовић Kilian Foth Ryathal CommunityBot | exact duplicate | |
| May 23, 2013 at 14:46 | review | Close votes | |||
| May 25, 2013 at 15:50 | |||||
| Feb 18, 2011 at 16:10 | comment | added | Armand | Might have been nice for your manager to try to understand the code before wasting your time. I'd review the code separately and then come back with queries and comments, and ask for fixes if required. Then repeat the process, ad nauseam. | |
| Feb 18, 2011 at 15:51 | comment | added | Graviton | @Ramhound, question updated to address your comment. | |
| Feb 18, 2011 at 15:50 | history | edited | Graviton | CC BY-SA 2.5 | added 302 characters in body |
| Feb 18, 2011 at 15:38 | answer | added | Havoc P | timeline score: 8 | |
| Feb 18, 2011 at 15:24 | answer | added | Bill | timeline score: 3 | |
| Feb 18, 2011 at 14:41 | comment | added | Ramhound | I will attempt to answer your question, in order to do that I need some context, what do you want your team members to get out of the code review? My only advice until I get time to answer your question in detail would be that, a developers peers should be reviewing the code, and you should help the developers on your team do that. I can tell you from experience as a new developer on a project, and the user just "expects it to work", and if it doesn't then that is a major problem Code Reviews do work if they are done correctly. | |
| Feb 18, 2011 at 14:25 | answer | added | Brandon DuRette | timeline score: 9 | |
| Feb 18, 2011 at 13:01 | answer | added | CashCow | timeline score: 0 | |
| Feb 18, 2011 at 10:05 | answer | added | Péter Török | timeline score: 5 | |
| S Feb 18, 2011 at 8:49 | history | suggested | blueberryfields | CC BY-SA 2.5 | grammar, spelling, and general style for readability |
| Feb 18, 2011 at 8:45 | review | Suggested edits | |||
| S Feb 18, 2011 at 8:49 | |||||
| Feb 18, 2011 at 8:26 | answer | added | user131 | timeline score: 1 | |
| Feb 18, 2011 at 8:23 | history | tweeted | twitter.com/#!/StackProgrammer/status/38513894498504704 | ||
| Feb 18, 2011 at 8:04 | answer | added | KeesDijk | timeline score: 19 | |
| Feb 18, 2011 at 7:51 | history | edited | Graviton | CC BY-SA 2.5 | added 199 characters in body |
| Feb 18, 2011 at 7:41 | answer | added | jmq | timeline score: 1 | |
| Feb 18, 2011 at 7:27 | history | edited | Graviton | CC BY-SA 2.5 | added 1 characters in body |
| Feb 18, 2011 at 7:21 | history | asked | Graviton | CC BY-SA 2.5 |