I just finished participating in the “Reseach Software Engineers” conference in Germany, deRSE19 . From the conference I took home 2 important messages, the first one I will share in this post (and the second to come soon!).
Alexandra Hill gave a talk about “The art of giving and receiving code reviews” which I found both interesting as well as useful! I will share with you here the summary, and then the slides of the talk.
- Code reviews increase code quality, reduce bugs and lead to better design decisions.
- A pull request is in principle a creation of the author. Like a musician writing a song, they have feelings of ownership over the PR and thus can be hurt when someone is too critical.
- As an author, try to lower your defensiveness.
- As a reviewer, try to raise the confidence of the author.
- When commenting on a code review (regardless of your role), be mindful of where your comment will fall in the “conflict vs. reward” scale.
This is the summary, but if you are interested for more you may find her slides here: https://www.alexandra-hill.com/talks/ or even a youtube recording of the same talk I heard here: https://www.youtube.com/watch?v=XY6eA2_2hOg&feature=youtu.be
This brings me to a request I have! Some of you know that I have been developing the package DrWatson which is a scientific project assistant. Besides contributions from my student @JonasIsensee I’ve written most of the code myself. The problem with this is that pretty much noone reviews my code.
Therefore, if you use DrWatson, or you are at least interested in it, please get in touch with me in order to review my pull requests. I am not asking you to contribute code (which if you want that would be amazing), I am only asking for someone to review my Pull Requests. This will help the code become more readable, and the API design decisions to be more sensible.