<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="elementToProof">
<b style="font-weight:normal">
<p dir="ltr" style="line-height:1.38;text-align:justify;margin-top:0pt;margin-bottom:0pt">
<span style="font-size: 12pt; font-family: Arial; color: rgb(0, 0, 0); font-weight: 400;">Hi again!</span></p>
<br>
<p dir="ltr" style="line-height:1.38;text-align:justify;margin-top:0pt;margin-bottom:0pt">
<span style="font-size: 12pt; font-family: Arial; color: rgb(0, 0, 0); font-weight: 400;">Thank you all for your interest in the discussion. Reading through your responses gave us a better understanding of how the code review process works in your project and
 shaped some future research ideas.</span></p>
<br>
<p dir="ltr" style="line-height:1.38;text-align:justify;margin-top:0pt;margin-bottom:0pt">
<span style="font-size: 12pt; font-family: Arial; color: rgb(0, 0, 0); font-weight: 400;">I have replied to your emails individually, but I think it’ll be helpful to share a summary of the conversations I had with everyone on the list.</span></p>
<br>
<p dir="ltr" style="line-height:1.38;text-align:justify;margin-top:0pt;margin-bottom:0pt">
<span style="font-size: 12pt; font-family: Arial; color: rgb(0, 0, 0); font-weight: 400;">As I previously shared, we observed that changes to build files are 2 to 4 times less likely to receive comments from reviewers compared to changes in source code or test
 files. This observation comes from analyzing over 7 million changes to files across projects. The comparison was made in different settings, including when the change affects only one, two, or all three types of files (source/test/build code).</span></p>
<br>
<p dir="ltr" style="line-height:1.38;text-align:justify;margin-top:0pt;margin-bottom:0pt">
<span style="font-size: 12pt; font-family: Arial; color: rgb(0, 0, 0); font-weight: 400;">We also found that when build changes do receive comments from reviewers, these comments are more likely to discuss potential bugs in the code when compared to source/test
 code. This is an observation we have made after closely analyzing 500 sampled comments (in the context of the build change and the discussion that follows the comment).</span></p>
<br>
<p dir="ltr" style="line-height:1.38;text-align:justify;margin-top:0pt;margin-bottom:0pt">
<span style="font-size: 12pt; font-family: Arial; color: rgb(0, 0, 0); font-weight: 400;">We aim to understand how build changes are reviewed, meaning we are looking for practices and policies for build change review that might be different from reviewing other
 code changes. We also want to know if there are any challenges that complicate the build change review process.</span></p>
<br>
<p dir="ltr" style="line-height:1.38;text-align:justify;margin-top:0pt;margin-bottom:0pt">
<span style="font-size: 12pt; font-family: Arial; color: rgb(0, 0, 0); font-weight: 400;">Our goal is to understand the development teams’ perception of these questions rather than to find a correct answer. As we are controlling for your level of experience
 with build systems, your participation will be a great contribution no matter how much experience you have with build code review.</span></p>
<br>
<p dir="ltr" style="line-height:1.38;text-align:justify;margin-top:0pt;margin-bottom:0pt">
<span style="font-size: 12pt; font-family: Arial; color: rgb(0, 0, 0); font-weight: 400;">So, if you find this discussion interesting, please do not hesitate to sign up using the link below.</span></p>
<br>
<p dir="ltr" style="line-height:1.38;text-align:justify;margin-top:0pt;margin-bottom:0pt">
<span style="font-size: 12pt; font-family: Arial; color: rgb(0, 0, 0); background-color: rgb(255, 255, 255); font-weight: 400;">Sign up link:
</span><a href="https://forms.gle/Yec6ZvCemNiLktwr5"><span style="font-size: 12pt; font-family: Arial; color: rgb(17, 85, 204); background-color: rgb(255, 255, 255); font-weight: 400; text-decoration: underline; text-decoration-skip-ink: none;">https://forms.gle/Yec6ZvCemNiLktwr5</span></a></p>
<br>
<p dir="ltr" style="line-height:1.38;text-align:justify;margin-top:0pt;margin-bottom:0pt">
<span style="font-size: 12pt; font-family: Arial; color: rgb(0, 0, 0); font-weight: 400;">Best,</span></p>
<p dir="ltr" style="line-height:1.38;text-align:justify;margin-top:0pt;margin-bottom:0pt">
<span style="font-size: 12pt; font-family: Arial; color: rgb(0, 0, 0); font-weight: 400;">Mattie</span></p>
</b></div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Development <development-bounces@qt-project.org> on behalf of Mattie Nejati <mahtab.nejati@uwaterloo.ca><br>
<b>Sent:</b> Wednesday, July 20, 2022 2:29 PM<br>
<b>To:</b> development@qt-project.org <development@qt-project.org><br>
<b>Subject:</b> [Development] Code Review of Build File Changes</font>
<div> </div>
</div>
<style type="text/css" style="display:none">
<!--
p
        {margin-top:0;
        margin-bottom:0}
-->
</style>
<div dir="ltr">
<div class="x_elementToProof" style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0); background-color:rgb(255,255,255)">
<b style="font-weight:normal">
<p dir="ltr" style="line-height:1.38; text-align:justify; margin-top:0pt; margin-bottom:0pt; font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0); background-color:rgb(255,255,255)">
<span style="font-size:12pt; font-family:Arial; color:rgb(0,0,0); font-weight:400"></span><br>
</p>
</b><b style="font-weight:normal">
<p dir="ltr" style="line-height:1.38; margin-top:0pt; margin-bottom:0pt"><span style="font-size:12pt; color:rgb(0,0,0); font-weight:400">Hi Developers,</span></p>
<br>
<p dir="ltr" style="line-height:1.38; margin-top:0pt; margin-bottom:0pt"><span style="font-size:12pt; color:rgb(0,0,0); font-weight:400">I’m Mattie, a Ph.D. student at the University of Waterloo and I’ve been studying the code review process of build files
 in Qt. For example, I’ve found that changes to build files are 2 to 4 times less likely to be discussed during code review than changes to source code or test files.</span></p>
<br>
<p dir="ltr" style="line-height:1.38; margin-top:0pt; margin-bottom:0pt"><span style="font-size:12pt; color:rgb(0,0,0); font-weight:400">As the next step in our study, we are recruiting developers for a 30-minute interview to help us shed light on:</span></p>
<br>
<p dir="ltr" style="line-height:1.38; margin-top:0pt; margin-bottom:0pt"><span style="font-size:12pt; color:rgb(0,0,0); font-weight:400">1. How build changes are reviewed?</span></p>
<p dir="ltr" style="line-height:1.38; margin-top:0pt; margin-bottom:0pt"><span style="font-size:12pt; color:rgb(0,0,0); font-weight:400">2. What are the challenges faced when reviewing build changes?</span></p>
<p dir="ltr" style="line-height:1.38; margin-top:0pt; margin-bottom:0pt"><span style="font-size:12pt; color:rgb(0,0,0); font-weight:400">3. Why is build change review different?</span></p>
<br>
<p dir="ltr" style="line-height:1.38; margin-top:0pt; margin-bottom:0pt"><span style="font-size:12pt; color:rgb(0,0,0); font-weight:400">Since the Qt review is featured in our analysis, we would appreciate it if you consider participating as an interviewee.
 If you are interested, please click the signup link below and follow the instructions. Once the study is concluded, we will share our findings with the Qt community first.</span></p>
<br>
</b></div>
<div><span style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0); background-color:rgb(255,255,255)">Signup link:
</span><a href="https://forms.gle/Yec6ZvCemNiLktwr5"><span style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; background-color:rgb(255,255,255)">https://forms.gle/Yec6ZvCemNiLktwr5</span></a><span style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0); background-color:rgb(255,255,255)"> </span></div>
<div class="x_elementToProof" style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0); background-color:rgb(255,255,255)">
<b style="font-weight:normal"><br>
<p dir="ltr" style="line-height:1.38; margin-top:0pt; margin-bottom:0pt"><span style="font-size:12pt; color:rgb(0,0,0); font-weight:400">Best,</span></p>
<p dir="ltr" style="line-height:1.38; margin-top:0pt; margin-bottom:0pt"><span style="font-size:12pt; color:rgb(0,0,0); font-weight:400">Mattie</span></p>
</b><br class="x_Apple-interchange-newline">
<br>
</div>
</div>
</body>
</html>