First step of Rails Contribute #shibuyarb

Copyright Drecom Co., Ltd. All Rights Reserved.

First step of Rails Contribute～Railsにプルリクを送ってみた～

@sue4452013/11/20 渋谷.rb


自己紹介

sue445 @drecom● 社内ツール系とかサーバ構築とか● 好きな言語はJava● TDDキチガイ（ティー・ワダ教）● 今期も来期もサザエさんとプリキュア推し● AZusaar!! , Chrome Gitlab Notifier とか

http://azusaar.appspot.com/

https://chrome.google.com/webstore/detail/chrome-gitlab-notifier/eageapgbnjicdjjihgclpclilenjbobi

http://azusaar.appspot.com/


今期の嫁

http://instagram.com/p/Z0RcwNg5Ep/




本妻

http://instagram.com/p/Z0RsGXg5E_/




ダブルピース

http://instagram.com/p/gyDzqLg5NE/




トリプルピース

http://instagram.com/p/gzG8JNg5HD/




キュア本棚

http://instagram.com/p/gahcRhg5Oa/




あじぇんだ

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


Ruby on Railsについて

説明不要

割愛

省略

ggrks


発端

ARでSELECT 〜 FOR UPDATEしたかった


やり方はすぐに見つかった

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






Sample Code

# select * from accounts where id=1 for updateAccount.find(1, lock: true)


Sample Code

# select * from accounts where id=1 for updateAccount.find(1, lock: true)

# => DEPRECATION WARNING: Passing options to #find is deprecated. Please build a scope and then call #find on it.


＿人人人人人人人人人人＿

＞　サンプルが非推奨　＜

￣ Y^Y^Y^Y^Y^Y^Y^Y^Y￣


あー、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.

http://edgeguides.rubyonrails.org/upgrading_ruby_on_rails.html




Chatter(社内SNS)で聞いたら


2分後にレスがきたｗｗｗ


lock! と thx! をかけています


せっかくなのでこんとりびゅーとしたかった


サンプルコードを1行直すだけでいい・・・


テスト書かなくていいから楽


PR送る前に読んだもの

● Contributing to Ruby on Rails○ 「6 Contributing to the Rails

Documentation」の辺りを重点的に● API Documentation Guidelines

http://edgeguides.rubyonrails.org/contributing_to_ruby_on_rails.html

http://edgeguides.rubyonrails.org/contributing_to_ruby_on_rails.html

http://edgeguides.rubyonrails.org/api_documentation_guidelines.html

http://edgeguides.rubyonrails.org/api_documentation_guidelines.html


俺氏のスペック

● 高校の英語はオールレッド● 大学入試も二次に英語の無い大学を受

験● TOEICスコア300くらい (10年前)


英語どうですか？


聞くまでもない

英語ダメです！(ｷﾘｯ


PR送る時

英語が最大の壁


英語は読むより書く方が難しい


役に立ったもの

● ChangeLog を支える英語● git - 英語コミットコメントに使えるオシャレフ

レーズ集 - Qiita [キータ] ● Google翻訳

○ 何も考えずに入れるとSVCメチャクチャなので翻訳されやすい日本語にするこ

○ GTO=Google Translation Optimize

https://gist.github.com/hayajo/3938098

https://gist.github.com/hayajo/3938098

http://qiita.com/ken_c_lo/items/4cb49f0fb74e8778804d



http://translate.google.co.jp/

http://translate.google.co.jp/


PR送ったったｗｗｗ

https://github.com/rails/rails/pull/12560




マサカリの嵐！


かと思ったらRails界隈は優しかった


優しいダメ出しはたくさんあった


[ci skip]のことを失念してた

コメントだけの修正の時はコミットメッセージに [ci skip] が必要だった ><


コードを修正したら2クエリに増えた


コードを修正したら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]]


細かい言い回しはよー分からん


Merge pull request !!!!!!!!!!!!!!!!!!!!!!!!!!!!


たった3行だけRailsに貢献できた


まとめ


こんな僕でも

なんとかなった


お前らも

なんとかなる


「プログラマはコードで語り合え」なんて幻想

Technology

First step of Rails Contribute #shibuyarb