First step of Rails Contribute‎ #shibuyarb
Upcoming SlideShare
Loading in...5
×
 

First step of Rails Contribute‎ #shibuyarb

on

  • 1,941 views

渋谷.rb[:20131120] で発表した資料です

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

Statistics

Views

Total Views
1,941
Views on SlideShare
1,836
Embed Views
105

Actions

Likes
8
Downloads
1
Comments
0

6 Embeds 105

http://sue445.hatenablog.com 75
https://twitter.com 17
https://www.chatwork.com 5
http://feedly.com 5
https://www.inoreader.com 2
https://kcw.kddi.ne.jp 1

Accessibility

Categories

Upload Details

Uploaded via as Adobe PDF

Usage Rights

© All Rights Reserved

Report content

Flagged as inappropriate Flag as inappropriate
Flag as inappropriate

Select your reason for flagging this presentation as inappropriate.

Cancel
  • Full Name Full Name Comment goes here.
    Are you sure you want to
    Your message goes here
    Processing…
Post Comment
Edit your comment

First step of Rails Contribute‎ #shibuyarb First step of Rails Contribute‎ #shibuyarb Presentation Transcript

  • 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 とか Copyright Drecom Co., Ltd. All Rights Reserved.
  • 今期の嫁 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 Rights Reserved.
  • Sample Code # select * from accounts where id=1 for update Account.find(1, lock: true) Copyright Drecom Co., Ltd. All Rights Reserved.
  • 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.
  • _人人人人人人人人人人_ > サンプルが非推奨 <  ̄ 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 methods which previously accepted "finder options" no longer do. Copyright Drecom Co., Ltd. All Rights Reserved.
  • 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 Guidelines Copyright Drecom Co., Ltd. All Rights Reserved.
  • 俺氏のスペック ● 高校の英語はオールレッド ● 大学入試も二次に英語の無い大学を受 験 ● 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メチャクチャな ので翻訳されやすい日本語にするこ ○ GTO=Google Translation Optimize Copyright Drecom Co., Ltd. All Rights Reserved.
  • 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 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.
  • 細かい言い回しはよー分からん 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.