First step of Rails Contribute‎ #shibuyarb

3,826 views

Published on

渋谷.rb[:20131120] で発表した資料です
http://www.zusaar.com/event/1707006

Published in: Technology
0 Comments
9 Likes
Statistics
Notes
  • Be the first to comment

No Downloads
Views
Total views
3,826
On SlideShare
0
From Embeds
0
Number of Embeds
627
Actions
Shares
0
Downloads
3
Comments
0
Likes
9
Embeds 0
No embeds

No notes for slide

First step of Rails Contribute‎ #shibuyarb

  1. 1. First step of Rails Contribute ~Railsにプルリクを送ってみた~ @sue445 2013/11/20 渋谷.rb Copyright Drecom Co., Ltd. All Rights Reserved.
  2. 2. 自己紹介 sue445 @drecom ● 社内ツール系とかサーバ構築とか ● 好きな言語はJava ● TDDキチガイ(ティー・ワダ教) ● 今期も来期もサザエさんとプリキュア推し ● AZusaar!! , Chrome Gitlab Notifier とか Copyright Drecom Co., Ltd. All Rights Reserved.
  3. 3. 今期の嫁 http://instagram.com/p/Z0RcwNg5Ep/ Copyright Drecom Co., Ltd. All Rights Reserved.
  4. 4. 本妻 http://instagram.com/p/Z0RsGXg5E_/ Copyright Drecom Co., Ltd. All Rights Reserved.
  5. 5. ダブルピース http://instagram.com/p/gyDzqLg5NE/ Copyright Drecom Co., Ltd. All Rights Reserved.
  6. 6. トリプルピース http://instagram.com/p/gzG8JNg5HD/ Copyright Drecom Co., Ltd. All Rights Reserved.
  7. 7. キュア本棚 http://instagram.com/p/gahcRhg5Oa/ Copyright Drecom Co., Ltd. All Rights Reserved.
  8. 8. あじぇんだ ● ● ● ● ● Ruby on Railsについて 発端 PR送る前 PR送る PR送った後 Copyright Drecom Co., Ltd. All Rights Reserved.
  9. 9. Ruby on Railsについて 説明不要 割愛 省略 ggrks Copyright Drecom Co., Ltd. All Rights Reserved.
  10. 10. 発端 ARでSELECT 〜 FOR UPDATEしたかった Copyright Drecom Co., Ltd. All Rights Reserved.
  11. 11. やり方はすぐに見つかった http://api.rubyonrails. org/classes/ActiveRecord/Locking/Pessimistic. html Copyright Drecom Co., Ltd. All Rights Reserved.
  12. 12. Sample Code # select * from accounts where id=1 for update Account.find(1, lock: true) Copyright Drecom Co., Ltd. All Rights Reserved.
  13. 13. Sample Code # select * from accounts where id=1 for update Account.find(1, lock: true) # => DEPRECATION WARNING: Passing options to #find is deprecated. Please build a scope and then call #find on it. Copyright Drecom Co., Ltd. All Rights Reserved.
  14. 14. _人人人人人人人人人人_ > サンプルが非推奨 <  ̄ Y^Y^Y^Y^Y^Y^Y^Y^Y ̄ Copyright Drecom Co., Ltd. All Rights Reserved.
  15. 15. あー、WEB+DB Vol.73で読んだわー Ruby on Rails Guides Rails 4.0 has deprecated the old-style hash based finder API. This means that methods which previously accepted "finder options" no longer do. Copyright Drecom Co., Ltd. All Rights Reserved.
  16. 16. Chatter(社内SNS)で聞いたら Copyright Drecom Co., Ltd. All Rights Reserved.
  17. 17. 2分後にレスがきたwww Copyright Drecom Co., Ltd. All Rights Reserved.
  18. 18. lock! と thx! をかけています Copyright Drecom Co., Ltd. All Rights Reserved.
  19. 19. せっかくなのでこんとり びゅーとしたかった Copyright Drecom Co., Ltd. All Rights Reserved.
  20. 20. サンプルコードを1行直すだけでいい・・・ Copyright Drecom Co., Ltd. All Rights Reserved.
  21. 21. テスト書かなくていいから楽 Copyright Drecom Co., Ltd. All Rights Reserved.
  22. 22. PR送る前に読んだもの ● Contributing to Ruby on Rails ○ 「6 Contributing to the Rails Documentation」 の辺りを重点的に ● API Documentation Guidelines Copyright Drecom Co., Ltd. All Rights Reserved.
  23. 23. 俺氏のスペック ● 高校の英語はオールレッド ● 大学入試も二次に英語の無い大学を受 験 ● TOEICスコア300くらい (10年前) Copyright Drecom Co., Ltd. All Rights Reserved.
  24. 24. 英語どうですか? Copyright Drecom Co., Ltd. All Rights Reserved.
  25. 25. 聞くまでもない 英語ダメです!(キリッ Copyright Drecom Co., Ltd. All Rights Reserved.
  26. 26. PR送る時 英語が最大の壁 Copyright Drecom Co., Ltd. All Rights Reserved.
  27. 27. 英語は読むより書く方が 難しい Copyright Drecom Co., Ltd. All Rights Reserved.
  28. 28. 役に立ったもの ● ChangeLog を支える英語 ● git - 英語コミットコメントに使えるオシャレフ レーズ集 - Qiita [キータ] ● Google翻訳 ○ 何も考えずに入れるとSVCメチャクチャな ので翻訳されやすい日本語にするこ ○ GTO=Google Translation Optimize Copyright Drecom Co., Ltd. All Rights Reserved.
  29. 29. PR送ったったwww https://github.com/rails/rails/pull/12560 Copyright Drecom Co., Ltd. All Rights Reserved.
  30. 30. マサカリの嵐! Copyright Drecom Co., Ltd. All Rights Reserved.
  31. 31. かと思ったらRails界隈は 優しかった Copyright Drecom Co., Ltd. All Rights Reserved.
  32. 32. 優しいダメ出しはたくさん あった Copyright Drecom Co., Ltd. All Rights Reserved.
  33. 33. [ci skip]のことを失念してた コメントだけの修正の時はコミットメッセージに [ci skip] が必要だった >< Copyright Drecom Co., Ltd. All Rights Reserved.
  34. 34. コードを修正したら2クエリに増えた Copyright Drecom Co., Ltd. All Rights Reserved.
  35. 35. コードを修正したら2クエリに増えた ● [Before] Message.find(1, lock: true) ○ SELECT "messages".* FROM "messages" WHERE "messages"."id" = $1 LIMIT 1 FOR UPDATE [["id", 1]] ● [After] Message.find(1).lock! ○ SELECT "messages".* FROM "messages" WHERE "messages"."id" = $1 LIMIT 1 [["id", 1]] ○ SELECT "messages".* FROM "messages" WHERE "messages"."id" = $1 LIMIT 1 FOR UPDATE [["id", 1]] ● [Adviced] Message.lock.find(1) ○ SELECT "messages".* FROM "messages" WHERE "messages"."id" = $1 LIMIT 1 FOR UPDATE [["id", 1]] Copyright Drecom Co., Ltd. All Rights Reserved.
  36. 36. 細かい言い回しはよー分からん Copyright Drecom Co., Ltd. All Rights Reserved.
  37. 37. Merge pull request !!!!!!!!!!!!!!!!!!!!!!!!!!!! Copyright Drecom Co., Ltd. All Rights Reserved.
  38. 38. たった3行だけRailsに貢献できた Copyright Drecom Co., Ltd. All Rights Reserved.
  39. 39. まとめ Copyright Drecom Co., Ltd. All Rights Reserved.
  40. 40. こんな僕でも なんとかなった Copyright Drecom Co., Ltd. All Rights Reserved.
  41. 41. お前らも なんとかなる Copyright Drecom Co., Ltd. All Rights Reserved.
  42. 42. 「プログラマはコードで語 り合え」なんて幻想 Copyright Drecom Co., Ltd. All Rights Reserved.

×