Ce diaporama a bien été signalé.
Nous utilisons votre profil LinkedIn et vos données d’activité pour vous proposer des publicités personnalisées et pertinentes. Vous pouvez changer vos préférences de publicités à tout moment.


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
• 

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

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

3 047 vues

Publié le

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

Publié dans : Logiciels
  • Soyez le premier à commenter

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

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

×