誰だって初めは怖いものさ。けど、嬉しいことだってある。
僕は、こんな感じ。*1
リマインダー
Custom reconnect_attempts to Fixnum by otoyo · Pull Request #622 · redis/redis-rb · GitHub
僕
レビューしてもらえませんか？
オーナー
7週間後にリマインドして 🌴 🌞
僕
わかった。良いバケーションを 🏝
僕
7週間経ったよ 🌞
オーナー
リマインドありがとう！週末だから、来週のTODOリストに入れておく！
--
これが文化の違いってやつか…！
最初はやっぱり怖かった
Mojolicious という Perl の Webアプリケーションフレームワークにて。
lookup X-Forwarded-Proto for SSL by otoyo · Pull Request #616 · mojolicious/mojo · GitHub
僕
ELB のために
X-Forwarded-Protoサポートして！
オーナー
で、
X-Forwarded-Protoは一般的なの？
僕
一般的かどうかはわからない…
ELBのヘッダは変えられないから、Mojolicious が
X-Forwarded-Protoを見てくれれば便利かなって。けど、不要ならクローズしてください。ありがとう。
オーナー
1つ言えるのは「Mojolicious にあらゆる
X-Forwarded-*ヘッダを詰め込みたくない(多すぎる！)」ってこと。我々は最も一般的なものだけサポートすべきだから、その他はCPANのプラグインでやって。
--
怒られつつも、結局このあと取り込んでもらえた。OSSって厳しいけど楽しい！
ほんの少しからはじめよう
fix dequeue for empty by otoyo · Pull Request #1217 · resque/resque · GitHub
僕
dequeueはキューがないとき 0 を期待しているけど、今は nil を返している。
オーナー
ここはこう書いた方が良いね。
僕
アドバイスありがとう！
オーナー
マージした。貢献ありがとう。
--
たった数行でも貢献は貢献。小さなPull Requestが少しずつOSSを良くしていくんだ。
時には断られることもある
僕
raise_errorsオプションを有効にしてもエラーハンドラがエラーをキャッチしちゃってる。思うに、この場合エラーハンドラを無視して raise すべきじゃないかな。
オーナー
それは正しいけど、すぐには変更できないんだ。
--
ポピュラーなプロダクトほど影響が大きいので、すぐには変更できないってこともある。
こればかりは仕方ない。
感謝されるとすごく嬉しい！
Support batch update resources by otoyo · Pull Request #344 · zendesk/zendesk_api_client_rb · GitHub
オーナー
(バージョン)1.15.0 🎉
僕
レビューありがとう！
オーナー
いくつもグレートなPRありがとう！
--
感謝されるとすごく嬉しい！さあ、Pull Requestしよう！
こちらもあわせてどうぞ
*1:読みやすいように意訳してあるから原文はリンク先を見てほしい