First step of Rails Contribute‎ #shibuyarb

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 Notifier とか


今期の嫁

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 update
Account.find(1, lock: true)


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.


＿人人人人人人人人人人＿
＞　サンプルが非推奨　＜
￣ 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.


Chatter(社内SNS)で聞いたら


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


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


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


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


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


PR送る前に読んだもの
● Contributing to Ruby on Rails
○ 「6 Contributing to the Rails
Documentation」の辺りを重点的に
● API Documentation Guidelines


俺氏のスペック

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


英語どうですか？


聞くまでもない

英語ダメです！(ｷﾘｯ


PR送る時

英語が最大の壁


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


役に立ったもの
● ChangeLog を支える英語
● git - 英語コミットコメントに使えるオシャレフ
レーズ集 - Qiita [キータ]
● Google翻訳
○ 何も考えずに入れるとSVCメチャクチャな
ので翻訳されやすい日本語にするこ
○ GTO=Google Translation Optimize


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!
"messages"."id" = $1 LIMIT 1 [["id", 1]]

● [Adviced] Message.lock.find(1)


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


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


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


まとめ


こんな僕でも
なんとかなった


お前らも
なんとかなる


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


First step of Rails Contribute‎ #shibuyarb

Recommended

Recommended

More Related Content

What's hot

What's hot (20)

Similar to First step of Rails Contribute‎ #shibuyarb

Similar to First step of Rails Contribute‎ #shibuyarb (17)

More from Go Sueyoshi (a.k.a sue445)

More from Go Sueyoshi (a.k.a sue445) (14)

Recently uploaded

Recently uploaded (9)

First step of Rails Contribute‎ #shibuyarb