On the optimal order of reading source code changes for review

Research output: Chapter in book/report/conference proceedingConference contributionResearchpeer review

Authors

Research Organisations

External Research Organisations

  • Universität Zürich (UZH)
View graph of relations

Details

Original languageEnglish
Title of host publicationProceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017
PublisherInstitute of Electrical and Electronics Engineers Inc.
Pages329-340
Number of pages12
ISBN (electronic)9781538609927
Publication statusPublished - 2 Nov 2017
Event2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017 - Shanghai, China
Duration: 17 Sept 201722 Sept 2017

Publication series

NameProceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017

Abstract

Change-based code review, e.g., in the form of pull requests, is the dominant style of code review in practice. An important option to improve review's efficiency is cognitive support for the reviewer. Nevertheless, review tools present the change parts under review sorted in alphabetical order of file path, thus leaving the effort of understanding the construction, connections, and logic of the changes on the reviewer. This leads to the question: How should a code review tool order the parts of a code change to best support the reviewer? We answer this question with a middle-range theory, which we generated inductively in a mixed methods study, based on interviews, an online survey, and existing findings from related areas. Our results indicate that an optimal order is mainly an optimal grouping of the change parts by relatedness. We present our findings as a collection of principles and formalize them as a partial order relation among review orders.

ASJC Scopus subject areas

Cite this

On the optimal order of reading source code changes for review. / Baum, Tobias; Schneider, Kurt; Bacchelli, Alberto.
Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017. Institute of Electrical and Electronics Engineers Inc., 2017. p. 329-340 8094433 (Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017).

Research output: Chapter in book/report/conference proceedingConference contributionResearchpeer review

Baum, T, Schneider, K & Bacchelli, A 2017, On the optimal order of reading source code changes for review. in Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017., 8094433, Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017, Institute of Electrical and Electronics Engineers Inc., pp. 329-340, 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017, Shanghai, China, 17 Sept 2017. https://doi.org/10.1109/ICSME.2017.28
Baum, T., Schneider, K., & Bacchelli, A. (2017). On the optimal order of reading source code changes for review. In Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017 (pp. 329-340). Article 8094433 (Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017). Institute of Electrical and Electronics Engineers Inc.. https://doi.org/10.1109/ICSME.2017.28
Baum T, Schneider K, Bacchelli A. On the optimal order of reading source code changes for review. In Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017. Institute of Electrical and Electronics Engineers Inc. 2017. p. 329-340. 8094433. (Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017). doi: 10.1109/ICSME.2017.28
Baum, Tobias ; Schneider, Kurt ; Bacchelli, Alberto. / On the optimal order of reading source code changes for review. Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017. Institute of Electrical and Electronics Engineers Inc., 2017. pp. 329-340 (Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017).
Download
@inproceedings{c13e632306ce45bb804206ea11a055d6,
title = "On the optimal order of reading source code changes for review",
abstract = "Change-based code review, e.g., in the form of pull requests, is the dominant style of code review in practice. An important option to improve review's efficiency is cognitive support for the reviewer. Nevertheless, review tools present the change parts under review sorted in alphabetical order of file path, thus leaving the effort of understanding the construction, connections, and logic of the changes on the reviewer. This leads to the question: How should a code review tool order the parts of a code change to best support the reviewer? We answer this question with a middle-range theory, which we generated inductively in a mixed methods study, based on interviews, an online survey, and existing findings from related areas. Our results indicate that an optimal order is mainly an optimal grouping of the change parts by relatedness. We present our findings as a collection of principles and formalize them as a partial order relation among review orders.",
author = "Tobias Baum and Kurt Schneider and Alberto Bacchelli",
note = "Publisher Copyright: {\textcopyright} 2017 IEEE. Copyright: Copyright 2018 Elsevier B.V., All rights reserved.; 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017 ; Conference date: 17-09-2017 Through 22-09-2017",
year = "2017",
month = nov,
day = "2",
doi = "10.1109/ICSME.2017.28",
language = "English",
series = "Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017",
publisher = "Institute of Electrical and Electronics Engineers Inc.",
pages = "329--340",
booktitle = "Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017",
address = "United States",

}

Download

TY - GEN

T1 - On the optimal order of reading source code changes for review

AU - Baum, Tobias

AU - Schneider, Kurt

AU - Bacchelli, Alberto

N1 - Publisher Copyright: © 2017 IEEE. Copyright: Copyright 2018 Elsevier B.V., All rights reserved.

PY - 2017/11/2

Y1 - 2017/11/2

N2 - Change-based code review, e.g., in the form of pull requests, is the dominant style of code review in practice. An important option to improve review's efficiency is cognitive support for the reviewer. Nevertheless, review tools present the change parts under review sorted in alphabetical order of file path, thus leaving the effort of understanding the construction, connections, and logic of the changes on the reviewer. This leads to the question: How should a code review tool order the parts of a code change to best support the reviewer? We answer this question with a middle-range theory, which we generated inductively in a mixed methods study, based on interviews, an online survey, and existing findings from related areas. Our results indicate that an optimal order is mainly an optimal grouping of the change parts by relatedness. We present our findings as a collection of principles and formalize them as a partial order relation among review orders.

AB - Change-based code review, e.g., in the form of pull requests, is the dominant style of code review in practice. An important option to improve review's efficiency is cognitive support for the reviewer. Nevertheless, review tools present the change parts under review sorted in alphabetical order of file path, thus leaving the effort of understanding the construction, connections, and logic of the changes on the reviewer. This leads to the question: How should a code review tool order the parts of a code change to best support the reviewer? We answer this question with a middle-range theory, which we generated inductively in a mixed methods study, based on interviews, an online survey, and existing findings from related areas. Our results indicate that an optimal order is mainly an optimal grouping of the change parts by relatedness. We present our findings as a collection of principles and formalize them as a partial order relation among review orders.

UR - http://www.scopus.com/inward/record.url?scp=85040538692&partnerID=8YFLogxK

U2 - 10.1109/ICSME.2017.28

DO - 10.1109/ICSME.2017.28

M3 - Conference contribution

AN - SCOPUS:85040538692

T3 - Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017

SP - 329

EP - 340

BT - Proceedings - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017

PB - Institute of Electrical and Electronics Engineers Inc.

T2 - 2017 IEEE International Conference on Software Maintenance and Evolution, ICSME 2017

Y2 - 17 September 2017 through 22 September 2017

ER -

By the same author(s)