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

Avatar for LINE Developers Taiwan

LINE Developers Taiwan PRO

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 維持合作品質