{"id":"https://openalex.org/W2624930143","doi":"https://doi.org/10.1109/ms.2017.265100500","title":"Code Reviewing in the Trenches: Challenges and Best Practices","display_name":"Code Reviewing in the Trenches: Challenges and Best Practices","publication_year":2017,"publication_date":"2017-06-16","ids":{"openalex":"https://openalex.org/W2624930143","doi":"https://doi.org/10.1109/ms.2017.265100500","mag":"2624930143"},"language":"en","primary_location":{"id":"doi:10.1109/ms.2017.265100500","is_oa":false,"landing_page_url":"https://doi.org/10.1109/ms.2017.265100500","pdf_url":null,"source":{"id":"https://openalex.org/S6725529","display_name":"IEEE Software","issn_l":"0740-7459","issn":["0740-7459","1937-4194"],"is_oa":false,"is_in_doaj":false,"is_core":true,"host_organization":"https://openalex.org/P4310320439","host_organization_name":"IEEE Computer Society","host_organization_lineage":["https://openalex.org/P4310320439","https://openalex.org/P4310319808"],"host_organization_lineage_names":["IEEE Computer Society","Institute of Electrical and Electronics Engineers"],"type":"journal"},"license":null,"license_id":null,"version":"publishedVersion","is_accepted":true,"is_published":true,"raw_source_name":"IEEE Software","raw_type":"journal-article"},"type":"article","indexed_in":["crossref"],"open_access":{"is_oa":false,"oa_status":"closed","oa_url":null,"any_repository_has_fulltext":false},"authorships":[{"author_position":"first","author":{"id":"https://openalex.org/A5069555336","display_name":"Laura MacLeod","orcid":"https://orcid.org/0000-0002-8543-294X"},"institutions":[{"id":"https://openalex.org/I4210164937","display_name":"Microsoft Research (United Kingdom)","ror":"https://ror.org/05k87vq12","country_code":"GB","type":"company","lineage":["https://openalex.org/I1290206253","https://openalex.org/I4210164937"]}],"countries":["GB"],"is_corresponding":false,"raw_author_name":"Laura MacLeod","raw_affiliation_strings":["Microsoft"],"raw_orcid":null,"affiliations":[{"raw_affiliation_string":"Microsoft","institution_ids":["https://openalex.org/I4210164937"]}]},{"author_position":"middle","author":{"id":"https://openalex.org/A5045376439","display_name":"Michaela Greiler","orcid":"https://orcid.org/0000-0002-2636-0560"},"institutions":[{"id":"https://openalex.org/I4210164937","display_name":"Microsoft Research (United Kingdom)","ror":"https://ror.org/05k87vq12","country_code":"GB","type":"company","lineage":["https://openalex.org/I1290206253","https://openalex.org/I4210164937"]}],"countries":["GB"],"is_corresponding":false,"raw_author_name":"Michaela Greiler","raw_affiliation_strings":["Microsoft"],"raw_orcid":null,"affiliations":[{"raw_affiliation_string":"Microsoft","institution_ids":["https://openalex.org/I4210164937"]}]},{"author_position":"middle","author":{"id":"https://openalex.org/A5038905934","display_name":"Margaret\u2010Anne Storey","orcid":"https://orcid.org/0000-0003-2278-2536"},"institutions":[{"id":"https://openalex.org/I212119943","display_name":"University of Victoria","ror":"https://ror.org/04s5mat29","country_code":"CA","type":"education","lineage":["https://openalex.org/I212119943"]}],"countries":["CA"],"is_corresponding":false,"raw_author_name":"Margaret-Anne Storey","raw_affiliation_strings":["University of Victoria"],"raw_orcid":null,"affiliations":[{"raw_affiliation_string":"University of Victoria","institution_ids":["https://openalex.org/I212119943"]}]},{"author_position":"middle","author":{"id":"https://openalex.org/A5036278905","display_name":"Christian Bird","orcid":"https://orcid.org/0000-0002-7774-0387"},"institutions":[{"id":"https://openalex.org/I4210164937","display_name":"Microsoft Research (United Kingdom)","ror":"https://ror.org/05k87vq12","country_code":"GB","type":"company","lineage":["https://openalex.org/I1290206253","https://openalex.org/I4210164937"]}],"countries":["GB"],"is_corresponding":false,"raw_author_name":"Christian Bird","raw_affiliation_strings":["Microsoft Research"],"raw_orcid":null,"affiliations":[{"raw_affiliation_string":"Microsoft Research","institution_ids":["https://openalex.org/I4210164937"]}]},{"author_position":"last","author":{"id":"https://openalex.org/A5008085180","display_name":"Jacek Czerwonka","orcid":null},"institutions":[{"id":"https://openalex.org/I4210164937","display_name":"Microsoft Research (United Kingdom)","ror":"https://ror.org/05k87vq12","country_code":"GB","type":"company","lineage":["https://openalex.org/I1290206253","https://openalex.org/I4210164937"]}],"countries":["GB"],"is_corresponding":false,"raw_author_name":"Jacek Czerwonka","raw_affiliation_strings":["Microsoft"],"raw_orcid":null,"affiliations":[{"raw_affiliation_string":"Microsoft","institution_ids":["https://openalex.org/I4210164937"]}]}],"institutions":[],"countries_distinct_count":2,"institutions_distinct_count":2,"corresponding_author_ids":[],"corresponding_institution_ids":[],"apc_list":{"value":2995,"currency":"USD","value_usd":2995},"apc_paid":null,"fwci":23.0219,"has_fulltext":false,"cited_by_count":141,"citation_normalized_percentile":{"value":0.99348795,"is_in_top_1_percent":true,"is_in_top_10_percent":true},"cited_by_percentile_year":{"min":90,"max":100},"biblio":{"volume":"35","issue":"4","first_page":"34","last_page":"42"},"is_retracted":false,"is_paratext":false,"is_xpac":false,"primary_topic":{"id":"https://openalex.org/T10260","display_name":"Software Engineering Research","score":0.9998999834060669,"subfield":{"id":"https://openalex.org/subfields/1710","display_name":"Information Systems"},"field":{"id":"https://openalex.org/fields/17","display_name":"Computer Science"},"domain":{"id":"https://openalex.org/domains/3","display_name":"Physical Sciences"}},"topics":[{"id":"https://openalex.org/T10260","display_name":"Software Engineering Research","score":0.9998999834060669,"subfield":{"id":"https://openalex.org/subfields/1710","display_name":"Information Systems"},"field":{"id":"https://openalex.org/fields/17","display_name":"Computer Science"},"domain":{"id":"https://openalex.org/domains/3","display_name":"Physical Sciences"}},{"id":"https://openalex.org/T11675","display_name":"Open Source Software Innovations","score":0.9986000061035156,"subfield":{"id":"https://openalex.org/subfields/1706","display_name":"Computer Science Applications"},"field":{"id":"https://openalex.org/fields/17","display_name":"Computer Science"},"domain":{"id":"https://openalex.org/domains/3","display_name":"Physical Sciences"}},{"id":"https://openalex.org/T10430","display_name":"Software Engineering Techniques and Practices","score":0.9973999857902527,"subfield":{"id":"https://openalex.org/subfields/1710","display_name":"Information Systems"},"field":{"id":"https://openalex.org/fields/17","display_name":"Computer Science"},"domain":{"id":"https://openalex.org/domains/3","display_name":"Physical Sciences"}}],"keywords":[{"id":"https://openalex.org/keywords/code-review","display_name":"Code review","score":0.8543672561645508},{"id":"https://openalex.org/keywords/best-practice","display_name":"Best practice","score":0.7156095504760742},{"id":"https://openalex.org/keywords/context","display_name":"Context (archaeology)","score":0.6409223079681396},{"id":"https://openalex.org/keywords/computer-science","display_name":"Computer science","score":0.6259180903434753},{"id":"https://openalex.org/keywords/process","display_name":"Process (computing)","score":0.5836154818534851},{"id":"https://openalex.org/keywords/code","display_name":"Code (set theory)","score":0.5253300666809082},{"id":"https://openalex.org/keywords/data-science","display_name":"Data science","score":0.5131385922431946},{"id":"https://openalex.org/keywords/trace","display_name":"TRACE (psycholinguistics)","score":0.43957677483558655},{"id":"https://openalex.org/keywords/software-engineering","display_name":"Software engineering","score":0.43749865889549255},{"id":"https://openalex.org/keywords/face","display_name":"Face (sociological concept)","score":0.42493438720703125},{"id":"https://openalex.org/keywords/software-development","display_name":"Software development","score":0.3427410125732422},{"id":"https://openalex.org/keywords/static-program-analysis","display_name":"Static program analysis","score":0.30625954270362854},{"id":"https://openalex.org/keywords/software","display_name":"Software","score":0.2854190468788147},{"id":"https://openalex.org/keywords/political-science","display_name":"Political science","score":0.1396266222000122},{"id":"https://openalex.org/keywords/sociology","display_name":"Sociology","score":0.1243143379688263},{"id":"https://openalex.org/keywords/programming-language","display_name":"Programming language","score":0.08501774072647095}],"concepts":[{"id":"https://openalex.org/C150292731","wikidata":"https://www.wikidata.org/wiki/Q1342704","display_name":"Code review","level":5,"score":0.8543672561645508},{"id":"https://openalex.org/C184356942","wikidata":"https://www.wikidata.org/wiki/Q830382","display_name":"Best practice","level":2,"score":0.7156095504760742},{"id":"https://openalex.org/C2779343474","wikidata":"https://www.wikidata.org/wiki/Q3109175","display_name":"Context (archaeology)","level":2,"score":0.6409223079681396},{"id":"https://openalex.org/C41008148","wikidata":"https://www.wikidata.org/wiki/Q21198","display_name":"Computer science","level":0,"score":0.6259180903434753},{"id":"https://openalex.org/C98045186","wikidata":"https://www.wikidata.org/wiki/Q205663","display_name":"Process (computing)","level":2,"score":0.5836154818534851},{"id":"https://openalex.org/C2776760102","wikidata":"https://www.wikidata.org/wiki/Q5139990","display_name":"Code (set theory)","level":3,"score":0.5253300666809082},{"id":"https://openalex.org/C2522767166","wikidata":"https://www.wikidata.org/wiki/Q2374463","display_name":"Data science","level":1,"score":0.5131385922431946},{"id":"https://openalex.org/C75291252","wikidata":"https://www.wikidata.org/wiki/Q1315756","display_name":"TRACE (psycholinguistics)","level":2,"score":0.43957677483558655},{"id":"https://openalex.org/C115903868","wikidata":"https://www.wikidata.org/wiki/Q80993","display_name":"Software engineering","level":1,"score":0.43749865889549255},{"id":"https://openalex.org/C2779304628","wikidata":"https://www.wikidata.org/wiki/Q3503480","display_name":"Face (sociological concept)","level":2,"score":0.42493438720703125},{"id":"https://openalex.org/C529173508","wikidata":"https://www.wikidata.org/wiki/Q638608","display_name":"Software development","level":3,"score":0.3427410125732422},{"id":"https://openalex.org/C137287247","wikidata":"https://www.wikidata.org/wiki/Q1329550","display_name":"Static program analysis","level":4,"score":0.30625954270362854},{"id":"https://openalex.org/C2777904410","wikidata":"https://www.wikidata.org/wiki/Q7397","display_name":"Software","level":2,"score":0.2854190468788147},{"id":"https://openalex.org/C17744445","wikidata":"https://www.wikidata.org/wiki/Q36442","display_name":"Political science","level":0,"score":0.1396266222000122},{"id":"https://openalex.org/C144024400","wikidata":"https://www.wikidata.org/wiki/Q21201","display_name":"Sociology","level":0,"score":0.1243143379688263},{"id":"https://openalex.org/C199360897","wikidata":"https://www.wikidata.org/wiki/Q9143","display_name":"Programming language","level":1,"score":0.08501774072647095},{"id":"https://openalex.org/C151730666","wikidata":"https://www.wikidata.org/wiki/Q7205","display_name":"Paleontology","level":1,"score":0.0},{"id":"https://openalex.org/C138885662","wikidata":"https://www.wikidata.org/wiki/Q5891","display_name":"Philosophy","level":0,"score":0.0},{"id":"https://openalex.org/C41895202","wikidata":"https://www.wikidata.org/wiki/Q8162","display_name":"Linguistics","level":1,"score":0.0},{"id":"https://openalex.org/C177264268","wikidata":"https://www.wikidata.org/wiki/Q1514741","display_name":"Set (abstract data type)","level":2,"score":0.0},{"id":"https://openalex.org/C199539241","wikidata":"https://www.wikidata.org/wiki/Q7748","display_name":"Law","level":1,"score":0.0},{"id":"https://openalex.org/C36289849","wikidata":"https://www.wikidata.org/wiki/Q34749","display_name":"Social science","level":1,"score":0.0},{"id":"https://openalex.org/C86803240","wikidata":"https://www.wikidata.org/wiki/Q420","display_name":"Biology","level":0,"score":0.0}],"mesh":[],"locations_count":1,"locations":[{"id":"doi:10.1109/ms.2017.265100500","is_oa":false,"landing_page_url":"https://doi.org/10.1109/ms.2017.265100500","pdf_url":null,"source":{"id":"https://openalex.org/S6725529","display_name":"IEEE Software","issn_l":"0740-7459","issn":["0740-7459","1937-4194"],"is_oa":false,"is_in_doaj":false,"is_core":true,"host_organization":"https://openalex.org/P4310320439","host_organization_name":"IEEE Computer Society","host_organization_lineage":["https://openalex.org/P4310320439","https://openalex.org/P4310319808"],"host_organization_lineage_names":["IEEE Computer Society","Institute of Electrical and Electronics Engineers"],"type":"journal"},"license":null,"license_id":null,"version":"publishedVersion","is_accepted":true,"is_published":true,"raw_source_name":"IEEE Software","raw_type":"journal-article"}],"best_oa_location":null,"sustainable_development_goals":[{"display_name":"Industry, innovation and infrastructure","score":0.5,"id":"https://metadata.un.org/sdg/9"}],"awards":[],"funders":[],"has_content":{"pdf":false,"grobid_xml":false},"content_urls":null,"referenced_works_count":11,"referenced_works":["https://openalex.org/W2020400699","https://openalex.org/W2036487649","https://openalex.org/W2048926275","https://openalex.org/W2075382909","https://openalex.org/W2091990486","https://openalex.org/W2139092060","https://openalex.org/W2148854374","https://openalex.org/W2152168421","https://openalex.org/W2168612058","https://openalex.org/W4231404931","https://openalex.org/W4235786747"],"related_works":["https://openalex.org/W4213125952","https://openalex.org/W3153702491","https://openalex.org/W2969257295","https://openalex.org/W3008981372","https://openalex.org/W2206096527","https://openalex.org/W2069798959","https://openalex.org/W2151979607","https://openalex.org/W4246032972","https://openalex.org/W2986478207","https://openalex.org/W2100259153"],"abstract_inverted_index":{"Code":[0,15],"review":[1,16,91],"has":[2],"been":[3],"widely":[4,68],"adopted":[5],"by":[6,34,73],"and":[7,12,36,47,70,101,135,142],"adapted":[8],"to":[9,81,99,105,128],"open":[10],"source":[11],"industrial":[13,40],"projects.":[14],"practices":[17,104],"have":[18,50,94],"undergone":[19],"extensive":[20],"research,":[21],"with":[22,44],"most":[23],"studies":[24],"relying":[25],"on":[26,54,85,112,157],"trace":[27],"data":[28],"from":[29,120],"tool":[30],"reviews,":[31],"sometimes":[32],"augmented":[33],"surveys":[35],"interviews.":[37],"Several":[38],"recent":[39,122],"research":[41],"studies,":[42],"along":[43],"blog":[45],"posts":[46],"white":[48],"papers,":[49],"revealed":[51],"additional":[52],"insights":[53,119],"code":[55,65,90],"reviewing":[56,66],"\u201cfrom":[57],"the":[58,61,74,113,130],"trenches.\u201d":[59],"Unfortunately,":[60],"lessons":[62],"learned":[63],"about":[64],"are":[67],"dispersed":[69],"poorly":[71],"summarized":[72],"existing":[75,87,114],"literature.":[76],"In":[77],"particular,":[78],"practitioners":[79,146],"wishing":[80],"adopt":[82,106],"or":[83,88],"reflect":[84],"an":[86],"new":[89],"process":[92],"might":[93],"difficulty":[95],"determining":[96],"what":[97],"challenges":[98,131],"expect":[100],"which":[102],"best":[103,139],"for":[107],"their":[108],"development":[109],"context.":[110],"Building":[111],"literature,":[115],"this":[116],"article":[117,150],"adds":[118],"a":[121,154],"large-scale":[123],"study":[124],"of":[125,153],"Microsoft":[126],"developers":[127],"summarize":[129],"that":[132,145],"code-change":[133],"authors":[134],"reviewers":[136],"face,":[137],"suggest":[138],"code-reviewing":[140],"practices,":[141],"discuss":[143],"tradeoffs":[144],"should":[147],"consider.":[148],"This":[149],"is":[151],"part":[152],"theme":[155],"issue":[156],"Process":[158],"Improvement.":[159]},"counts_by_year":[{"year":2026,"cited_by_count":7},{"year":2025,"cited_by_count":10},{"year":2024,"cited_by_count":26},{"year":2023,"cited_by_count":13},{"year":2022,"cited_by_count":24},{"year":2021,"cited_by_count":25},{"year":2020,"cited_by_count":17},{"year":2019,"cited_by_count":17},{"year":2018,"cited_by_count":1},{"year":2017,"cited_by_count":1}],"updated_date":"2026-08-21T09:56:20.448147","created_date":"2025-10-10T00:00:00"}