How to review code effectively: A GitHub staff engineer’s philosophy
As a staff engineer at GitHub, code review is one of my main focus areas in my day to day work. Over the past eight years, I’ve reviewed more than 7,000 pull requests. Why so many? Because code review is crucial to building good software and another set of eyes can often spot issues you would have otherwise missed. I see code review as one of the most important aspects of my job. In fact, whenever
コードレビューã®ã‚¯ã‚ªãƒªãƒ†ã‚£ã¨ã‚¹ãƒ”ード，ã¨ãã«ã‚¹ãƒ”ードã«ã¤ã„ã¦ï¼Œãã‚Œã¨ã‚³ãƒŸãƒ¥ãƒ‹ã‚±ãƒ¼ã‚·ãƒ§ãƒ³ã«ã¤ã„㦠- hitode909ã®æ—¥è¨˜
ソフトウェアを作るã¨ãã«ã‚¯ã‚ªãƒªãƒ†ã‚£ã¨ã‚¹ãƒ”ードã®ãƒãƒ©ãƒ³ã‚¹ã‚’ã¨ã‚ŠãŸãã¦ï¼Œã©ã¡ã‚‰ã‹ã«åã£ã¦ã¯ã„ã‘ãªã，ã©ã¡ã‚‰ã‚‚ã‚ープã—ãªã„ã¨ã„ã‘ãªã„．ã™ã”ã雑ã«*1ã¨ã‚‰ãˆã‚‹ã¨ï¼Œ クオリティ→æ£ã—ãå‹•ã，ä¸å…·åˆãŒãªã„ã»ã†ãŒã‚ˆã„ スピード→(計算時間ã§ã¯ãªã)æ—©ã作れるã»ã†ãŒã‚ˆã„ ã¨ã„ã†ã“ã¨ã«ãªã‚‹ï¼Ž コードレビューã§ã¯ï¼Œä¸å…·åˆã‚’見ã¤ã‘ã¦ç›´ã—ã¦ã‚‚らã£ãŸã‚Šï¼Œå‹•ãã¯ã—ã¦ã‚‚コードã®å¯èªæ€§ã«å•é¡ŒãŒã‚ã£ã¦ç›´ã—ã¦ã‚‚らã£ãŸã‚Šã¨ï¼Œã‚¯ã‚ªãƒªãƒ†ã‚£ã«ç›®ã‚’å‘ã‘られãŒã¡ã ã¨æ€ã†ï¼Ž ã¨ã“ã‚ã§ï¼Œã‚³ãƒ¼ãƒ‰ãƒ¬ãƒ“ューã¨ã‚¹ãƒ”ードã®é–¢ã‚ã‚Šã«ã¤ã„ã¦è€ƒãˆã¦ã¿ã‚‹ï¼Žã‚¹ãƒ”ードã®ãŸã‚ã«ã§ãã‚‹ã“ã¨ã¯ã„ãã¤ã‹ã‚り， æ—©ãèªã¿å§‹ã‚る→他ã®ã“ã¨ã‚„ã£ã¦ã¦ã‚‚手をæ¢ã‚ã¦èªã¿å§‹ã‚ãŸã‚Šï¼Œ1æ—¥ã®ã†ã¡æ±ºã¾ã£ãŸæ™‚é–“ã«ãƒ¬ãƒ“ュータイムをè¨ã‘ãŸã‚Š 速ãèªã‚€â†’ã“ã‚Œã¯ã‚³ãƒ„ã¨ã‹ã‚ã‚‹*2ã‘ã©ç²¾èªã—ãªã„ã¨ã„ã‘ãªã„ã®ã§é›£ã—ã„ ä¸å…·åˆã‚’見逃ã•ãªã„→リリース後ã¨ã‹ï¼Œãƒªãƒªãƒ¼ã‚¹ç›´å‰ã«æ£ã—ãå‹•ã‹ãªã„ã“ã¨ãŒåˆ†ã‹ã£ãŸã‚‰å¤§ããªæ‰‹
コードレビュー - hitode909ã®æ—¥è¨˜
コードレビュー，慣れるã¨ã§ãã‚‹ã‘ã©ï¼Œã„ããªã‚Šdiffを渡ã•ã‚Œã¦ï¼Œã©ã†ãžè¦‹ã¦ãã ã•ã„ã¨è¨€ã‚ã‚Œã¦ã‚‚よãã‚ã‹ã‚‰ãªã„ã¨æ€ã†ï¼Ž ã‚„ã‚Šã¾ã—ょã†ã¨ã„ã†ã®ã¯ã„ã„ã‘ã©ï¼ŒãŸã むやã¿ã«èªã‚“ã§ã‚‚ã†ã¾ãã„ã‹ãªã„．変更ãŒã‚る程度大ãããªã‚‹ã¨diffã ã‘見ã¦ã‚‚よãã‚ã‹ã‚‰ãªã„ã®ã§ï¼Œã„ã‚ã„ã‚見るã“ã¨ã«ãªã‚‹ï¼Ž 僕ã¯ã„ã¤ã‚‚以下ã®ã‚ˆã†ãªã“ã¨ã‚’ç„¡æ„è˜ã«ã‚„ã£ã¦ã¦ï¼Œã†ã¾ãã„ã£ã¦ã‚‹æ°—ãŒã—ã¦ã‚‹ï¼ŽGitHubã®Pull Requestã®ä»•çµ„ã¿ã‚’使ã£ã¦ã‚‹å‰æã§ï¼Ž Discussionã‚’ã•ã‚‰ã£ã¨çœºã‚ã¦ã©ã†ã„ã†å•é¡Œã‚’解決ã—ãŸã„ã®ã‹è¦‹ã‚‹ Commit Statusを見ã¦ï¼Œãƒ†ã‚¹ãƒˆé€šã£ã¦ã‚‹ã“ã¨ã‚’確èªã™ã‚‹ Commitsタブã§1コミットãšã¤ãƒ–ラウザã®æ–°ã—ã„タブã«é–‹ã 全部クリックã—終ã‚ã£ãŸã‚‰å¤ã„é †ã«1コミットãšã¤èªã‚€ æ°—ã«ãªã‚‹ç‚¹ãŒã‚ã£ãŸã‚‰ã‚¨ãƒ‡ã‚£ã‚¿ã¨ã‹ã«ãƒ¡ãƒ¢ã—ã¦ãŠã．ã‚ã¨ã§æ›¸ãç›´ã•ã‚Œã‚‹ã‹ã‚‚ã—ã‚Œãªã„ã®ã§ï¼Œã¾ã コメントã—ãªã„ 全コミット見終ã‚ã£ãŸã‚‰Files
入社ã‹ã‚‰ã®åŠå¹´é–“ã§ã‚³ãƒ¼ãƒ‰ãƒ¬ãƒ“ューã§æŒ‡æ‘˜ã•ã‚ŒãŸã“ã¨ã®ã¾ã¨ã‚ - 30æ³ã‹ã‚‰ã®ãƒ—ãƒã‚°ãƒ©ãƒŸãƒ³ã‚°
実務未経験ã§ãƒ—ãƒã‚°ãƒ©ãƒžã¨ã—ã¦å…¥ç¤¾ã—ã¦åŠå¹´ä»¥ä¸ŠãŒçµŒã£ãŸã€‚ コードレビューã§æŒ‡æ‘˜ã•ã‚ŒãŸã“ã¨ã‚’備忘録ã¨ã—ã¦ã¾ã¨ã‚ã¦ãŠã。 自分ãªã‚Šã«ã¾ã¨ã‚ãŸã‚‚ã®ãªã®ã§ã€ãƒ¬ãƒ“ュアーãŒè¨€ã„ãŸã‹ã£ãŸã“ã¨ã¨ãƒ‹ãƒ¥ã‚¢ãƒ³ã‚¹ã‚„解釈ãŒãšã‚Œã¦ã„ã‚‹å¯èƒ½æ€§ã¯ã‚る。 åˆæ©çš„ãªå†…容ã°ã‹ã‚Šã§æˆ‘ãªãŒã‚‰ã†ã‚“ã–ã‚Šã™ã‚‹ã€‚ ã›ã£ã‹ã優秀ãªåŒåƒšã°ã‹ã‚Šãªã®ã ã‹ã‚‰ã‚‚ã£ã¨é«˜åº¦ãªã“ã¨ã‚’å¦ã³ãŸã„ãŒã€ã“ã†ã„ã†åˆæ©çš„ãªã“ã¨ãŒå‡ºæ¥ãªã„ã®ãŒä¿ºã®ç¾çŠ¶ãªã®ã ã‹ã‚‰ã€ä»•æ–¹ãªã„。 ãã‚‚ãã‚‚PullRequestã‚’é€ã£ãŸã“ã¨ã‚‚ãªã‹ã£ãŸã‚ã‘ã ã—。入社åˆæ—¥ã¯ã€ä¸€äººã§PullRequestã®å‡ºã—方を練習ã—ã¦ã„ãŸã€‚ ãれを考ãˆã‚Œã°ã¾ã‚ã€ã“ã‚“ãªã‚‚ã®ã ã‚ã†ã‹ã€‚ 当ãŸã‚Šå‰ã®ã“ã¨ã‚’ã¡ã‚ƒã‚“ã¨å½“ãŸã‚Šå‰ã«å‡ºæ¥ã‚‹ã‚ˆã†ã«ãªã£ã¦ã€æ—©ãã€æ¬¡ã®ã‚¹ãƒ†ãƒ¼ã‚¸ã«é€²ã¿ãŸã„。 PullRequest(PR) PRã®ã‚¿ã‚¤ãƒˆãƒ«ã¯åˆ†ã‹ã‚Šã‚„ã™ã„ã‚‚ã®ã«ã€‚å¿…è¦ã«å¿œã˜ã¦ãƒã‚±ãƒƒãƒˆã®ç•ªå·ãªã©ã‚‚入れる。 コミットやPRã¯å‡ºæ¥ã‚‹ã ã‘粒度を細ã‹ãã™
1
リリースã€éšœå®³æƒ…å ±ãªã©ã®ã‚µãƒ¼ãƒ“スã®ãŠçŸ¥ã‚‰ã›
最新ã®äººæ°—エントリーã®é…ä¿¡
j次ã®ãƒ–ックマーク
kå‰ã®ãƒ–ックマーク
lã‚ã¨ã§èªã‚€
eコメント一覧を開ã
oページを開ã
{{#tags}}- {{label}}
{{/tags}}