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.
First step of Rails Contribute
~Railsにプルリクを送ってみた~

@sue445
2013/11/20 渋谷.rb
Copyright Drecom Co., Ltd. All Rights Reserved...
自己紹介
sue445 @drecom
● 社内ツール系とかサーバ構築とか
● 好きな言語はJava
● TDDキチガイ(ティー・ワダ教)
● 今期も来期もサザエさんとプリキュア推し
● AZusaar!! , Chrome Gitlab No...
今期の嫁

http://instagram.com/p/Z0RcwNg5Ep/
Copyright Drecom Co., Ltd. All Rights Reserved.
本妻

http://instagram.com/p/Z0RsGXg5E_/
Copyright Drecom Co., Ltd. All Rights Reserved.
ダブルピース

http://instagram.com/p/gyDzqLg5NE/
Copyright Drecom Co., Ltd. All Rights Reserved.
トリプルピース

http://instagram.com/p/gzG8JNg5HD/
Copyright Drecom Co., Ltd. All Rights Reserved.
キュア本棚

http://instagram.com/p/gahcRhg5Oa/
Copyright Drecom Co., Ltd. All Rights Reserved.
あじぇんだ
●
●
●
●
●

Ruby on Railsについて
発端
PR送る前
PR送る
PR送った後

Copyright Drecom Co., Ltd. All Rights Reserved.
Ruby on Railsについて

説明不要
割愛
省略
ggrks
Copyright Drecom Co., Ltd. All Rights Reserved.
発端

ARでSELECT 〜 FOR
UPDATEしたかった

Copyright Drecom Co., Ltd. All Rights Reserved.
やり方はすぐに見つかった

http://api.rubyonrails.
org/classes/ActiveRecord/Locking/Pessimistic.
html

Copyright Drecom Co., Ltd. All R...
Sample Code
# select * from accounts where id=1 for update
Account.find(1, lock: true)

Copyright Drecom Co., Ltd. All Rig...
Sample Code
# select * from accounts where id=1 for update
Account.find(1, lock: true)

# => DEPRECATION WARNING: Passing
...
_人人人人人人人人人人_
> サンプルが非推奨 <
 ̄ Y^Y^Y^Y^Y^Y^Y^Y^Y ̄

Copyright Drecom Co., Ltd. All Rights Reserved.
あー、WEB+DB Vol.73で読んだわー
Ruby on Rails Guides
Rails 4.0 has deprecated the old-style hash
based finder API. This means that ...
Chatter(社内SNS)で聞いたら

Copyright Drecom Co., Ltd. All Rights Reserved.
2分後にレスがきたwww

Copyright Drecom Co., Ltd. All Rights Reserved.
lock! と thx! をかけています

Copyright Drecom Co., Ltd. All Rights Reserved.
せっかくなのでこんとり
びゅーとしたかった

Copyright Drecom Co., Ltd. All Rights Reserved.
サンプルコードを1行直すだけでいい・・・

Copyright Drecom Co., Ltd. All Rights Reserved.
テスト書かなくていいから楽

Copyright Drecom Co., Ltd. All Rights Reserved.
PR送る前に読んだもの
● Contributing to Ruby on Rails
○ 「6 Contributing to the Rails
Documentation」 の辺りを重点的に
● API Documentation Gui...
俺氏のスペック

● 高校の英語はオールレッド
● 大学入試も二次に英語の無い大学を受
験
● TOEICスコア300くらい (10年前)

Copyright Drecom Co., Ltd. All Rights Reserved.
英語どうですか?

Copyright Drecom Co., Ltd. All Rights Reserved.
聞くまでもない

英語ダメです!(キリッ

Copyright Drecom Co., Ltd. All Rights Reserved.
PR送る時

英語が最大の壁

Copyright Drecom Co., Ltd. All Rights Reserved.
英語は読むより書く方が
難しい

Copyright Drecom Co., Ltd. All Rights Reserved.
役に立ったもの
● ChangeLog を支える英語
● git - 英語コミットコメントに使えるオシャレフ
レーズ集 - Qiita [キータ]
● Google翻訳
○ 何も考えずに入れるとSVCメチャクチャな
ので翻訳されやすい日本語にす...
PR送ったったwww
https://github.com/rails/rails/pull/12560

Copyright Drecom Co., Ltd. All Rights Reserved.
マサカリの嵐!

Copyright Drecom Co., Ltd. All Rights Reserved.
かと思ったらRails界隈は
優しかった

Copyright Drecom Co., Ltd. All Rights Reserved.
優しいダメ出しはたくさん
あった

Copyright Drecom Co., Ltd. All Rights Reserved.
[ci skip]のことを失念してた
コメントだけの修正の時はコミットメッセージに [ci
skip] が必要だった ><

Copyright Drecom Co., Ltd. All Rights Reserved.
コードを修正したら2クエリに増えた

Copyright Drecom Co., Ltd. All Rights Reserved.
コードを修正したら2クエリに増えた
● [Before] Message.find(1, lock: true)
○ SELECT "messages".* FROM "messages" WHERE
"messages"."id" = $1 ...
細かい言い回しはよー分からん

Copyright Drecom Co., Ltd. All Rights Reserved.
Merge pull request !!!!!!!!!!!!!!!!!!!!!!!!!!!!

Copyright Drecom Co., Ltd. All Rights Reserved.
たった3行だけRailsに貢献できた

Copyright Drecom Co., Ltd. All Rights Reserved.
まとめ

Copyright Drecom Co., Ltd. All Rights Reserved.
こんな僕でも
なんとかなった

Copyright Drecom Co., Ltd. All Rights Reserved.
お前らも
なんとかなる

Copyright Drecom Co., Ltd. All Rights Reserved.
「プログラマはコードで語
り合え」なんて幻想

Copyright Drecom Co., Ltd. All Rights Reserved.
Upcoming SlideShare
Loading in …5
×

First step of Rails Contribute‎ #shibuyarb

4,203 views

Published on

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

Published in: Technology
  • Dating for everyone is here: ❤❤❤ http://bit.ly/36cXjBY ❤❤❤
       Reply 
    Are you sure you want to  Yes  No
    Your message goes here
  • Sex in your area is here: ♥♥♥ http://bit.ly/36cXjBY ♥♥♥
       Reply 
    Are you sure you want to  Yes  No
    Your message goes here

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.

×