Upgrade to Pro — share decks privately, control downloads, hide ads and more …

發PR被同事「留言灌爆」! 他只用這2方法解決,同事驚:輕鬆很多

發PR被同事「留言灌爆」! 他只用這2方法解決,同事驚:輕鬆很多

活動:LINE Developers Meetup 開發者小聚 #18 - TECH FRESH 畢業展
活動頁面:https://linegroup.kktix.cc/events/20230627-tech-fresh-gratudation
講者:Ricky Hu

LINE Developers Taiwan

July 04, 2023
Tweet

More Decks by LINE Developers Taiwan

Other Decks in Technology

Transcript

  1. Ricky Hu TECH FRESH @ Android team LINE SHOPPING App

    NTUT EECS (graduated) ※Source from︓https://www.flaticon.com/free-icon/graduation-hat_1344761
  2. 13% 46% 17% 11% 13% 我被 Comment 了什麼? 在會議中討論 /

    私下討論 寫到相關的 Code 才知道要⽤ 少⾒的 Comment Spec Quality Conventions Reuse Others
  3. Tip 1: 預先閱讀 Coding Style Docs 1. 熟悉團隊的 Coding Convention

    Dart 語⾔官⽅推薦寫法:Effective Dart ※Source from︓https://github.com/Genies-des-mathematiques/sususudoku/pull/12
  4. Tip 2: 觀察其他 PR 的 Comment 1. 熟悉團隊的 Coding Convention

    Ricky 在 PR 留⾔ 哈密瓜發 PR 🍈 草莓在 PR 留⾔ 🍓 哈密瓜 修正留⾔ 🍈 Ricky & 草莓 留⾔ LGTM 🍓 哈密瓜 Merge PR 🍈 不知道留⾔什麼🤔 觀察別⼈的留⾔ 👀 ※Source from︓ https://www.flaticon.com/free-icon/pull-request_10824098 https://www.flaticon.com/free-icon/git_10042189 https://www.flaticon.com/free-icon/comment_3193015 https://www.flaticon.com/free-icon/hammer_7012680
  5. Big PR +589 -196 133 comments … 2. ⼤ PR

    分成多個⼩ PR PR 上的留⾔修⼀下! 對不起,我只是⼀個實習⽣… 😵💫 同事 😓 我
  6. Small PR #1 +223 -98 47 comments Small PR #3

    +196 -102 15 comments Small PR #2 +261 -144 24 comments 2. ⼤ PR 分成多個⼩ PR … LGTM! Merged! 😎 同事 😉 我
  7. ⼀點⼩⼩的想法 PR 被留⾔灌爆是⼀件好事? • 透過 PR 快速了解團隊 Convention • 團隊成員對於專案品質要求⾼

    實習⼩⼩⼼得 • TECH FRESH Mentor 學習指引 • Workshop、Team Building 維持合作品質