Successfully reported this slideshow.
We use your LinkedIn profile and activity data to personalize ads and to show you more relevant ads. You can change your ad preferences anytime.


Yasuhiro Kiyota @yasuhiroki
• Yasuhiro Kiyota
• @yasuhiroki
• Twitter: @duck_yasuhiroki
• GitHub: https://github.com/yasuhiroki
• Jenkins - Git Plugin...
https://certificates.cloudbees.com/10203903
$ echo "#code_kaizen" | grep -o . | sed 1d | tr '_' '|' | gsed -e '1i+-+' -e 's/./|&|/' -e '$a+-+'


Agenda
• 2
• 2 2 3
2
•
•
•
• Android
•
• AWS RDS + AWS Lambda + AWS API Gateway 

•
•
•
•
• Android
•
•
•
•
•
•
•
• 



1
•
• ← 



1
•
•
•
•
•
•
•
•
•
•
•
•
•
•
•
•
• Slack
•
• .github/PULL_REQUEST_TEMPLATE.md
•
•
•
•
•
• ≒
• …
• …
•
•
•
•
• PM or
•
•
•
• WIP / PleaseReview
• 

• PleaseReview 

• 43 / 51 (68%) 

1~3 Merge or Close

※ Please Review 

( )
• 

-> -> -> 

1~3
2 2 3
•
• Android
•
•
•
•
•
•
•
•
•
• 2 ( )
• 2 (New)
•
•
• 2
• 2 3
•
• Android
•
•
•
•
•
•
1
•
• +1 *2
•
•
• OK OK
•
null
OK
2
• 2 (NEW)
•
•
• …
• ->
• Git …
• -> SCM
• push
• -> remote
•
•
•
•
• …
• …
•
•
•
•
•
• Slack
•
•
• GitHub
• 

GitHub 

•
•
• CI 

…
• 2
•
• 

•
•
•
• 2 (NEW) PR 

49/61(80%) 1
• 1 30
•
• Git
• 1
• 2 ( ) PR 

80/107(74%) 1
• 

• …
•
•
•
•
•
•
コードレビューの文化を手探りで作っていった話
コードレビューの文化を手探りで作っていった話
コードレビューの文化を手探りで作っていった話
コードレビューの文化を手探りで作っていった話
コードレビューの文化を手探りで作っていった話
コードレビューの文化を手探りで作っていった話
コードレビューの文化を手探りで作っていった話
コードレビューの文化を手探りで作っていった話
コードレビューの文化を手探りで作っていった話
コードレビューの文化を手探りで作っていった話
コードレビューの文化を手探りで作っていった話
コードレビューの文化を手探りで作っていった話
コードレビューの文化を手探りで作っていった話
コードレビューの文化を手探りで作っていった話
Upcoming SlideShare
Loading in …5
×

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

3,046 views

Published on

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

Published in: Software
  • Be the first to comment

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

  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. • … • • • • • •

×