Successfully reported this slideshow.
Your SlideShare is downloading. ×

コードレビューの文化を手探りで作っていった話

Ad
Ad
Ad
Ad
Ad
Ad
Ad
Ad
Ad
Ad
Ad
Upcoming SlideShare
Menggambar proyeksi
Menggambar proyeksi
Loading in …3
×

Check these out next

1 of 55 Ad

コードレビューの文化を手探りで作っていった話

Download to read offline

#code_kaizen
コード改善 #2 で発表したスライドです。

SpeakerDeck: https://speakerdeck.com/yasuhiroki/kodorebiyunowen-hua-woshou-tan-ridezuo-tuteitutahua

#code_kaizen
コード改善 #2 で発表したスライドです。

SpeakerDeck: https://speakerdeck.com/yasuhiroki/kodorebiyunowen-hua-woshou-tan-ridezuo-tuteitutahua

Advertisement
Advertisement

More Related Content

Recently uploaded (20)

Advertisement

コードレビューの文化を手探りで作っていった話

  1. 1. 
 Yasuhiro Kiyota @yasuhiroki
  2. 2. • Yasuhiro Kiyota • @yasuhiroki • Twitter: @duck_yasuhiroki • GitHub: https://github.com/yasuhiroki • Jenkins - Git Plugin • http://qiita.com/yasuhiroki/items/61a2be613fc7dcfc8682 • Tips Golf/ • http://qiita.com/yasuhiroki/items/fad876db9e5505fceb03
  3. 3. https://certificates.cloudbees.com/10203903
  4. 4. $ echo "#code_kaizen" | grep -o . | sed 1d | tr '_' '|' | gsed -e '1i+-+' -e 's/./|&|/' -e '$a+-+'
  5. 5.
  6. 6. Agenda • 2 • 2 2 3
  7. 7. 2
  8. 8. • • • • Android • • AWS RDS + AWS Lambda + AWS API Gateway 

  9. 9. • • • • • Android • • •
  10. 10.
  11. 11. • •
  12. 12. • • 
 
 1
  13. 13. • • ← 
 
 1
  14. 14. • • • • • •
  15. 15. • • • • • •
  16. 16. • • • • • Slack •
  17. 17. • .github/PULL_REQUEST_TEMPLATE.md • • • • • • ≒ • … • … • •
  18. 18. • • • PM or • • •
  19. 19. • WIP / PleaseReview
  20. 20. • 
 • PleaseReview 

  21. 21. • 43 / 51 (68%) 
 1~3 Merge or Close
 ※ Please Review 
 ( ) • 
 -> -> -> 
 1~3
  22. 22. 2 2 3
  23. 23. • • Android • • • • •
  24. 24. • • • • • 2 ( ) • 2 (New) • •
  25. 25. • 2 • 2 3 • • Android
  26. 26.
  27. 27.
  28. 28.
  29. 29.
  30. 30. • •
  31. 31. 1 • • +1 *2 • • • OK OK • null OK
  32. 32. 2 • 2 (NEW) • • • … • -> • Git … • -> SCM • push • -> remote
  33. 33. • • • • • … • …
  34. 34. • • • • • • Slack • •
  35. 35. • GitHub • 
 GitHub 
 •
  36. 36. • • CI 
 … • 2 • • 

  37. 37. • • •
  38. 38. • 2 (NEW) PR 
 49/61(80%) 1 • 1 30 • • Git • 1
  39. 39. • 2 ( ) PR 
 80/107(74%) 1
  40. 40. • 

  41. 41. • … • • • • • •

×