Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
OSS初心者がつまづきながらOSSマナーを学んでいく話
Search
fuqda
July 05, 2019
Programming
4
3.2k
OSS初心者がつまづきながらOSSマナーを学んでいく話
Tama Ruby会議01の発表資料です。
fuqda
July 05, 2019
Tweet
Share
More Decks by fuqda
See All by fuqda
Composition API時代の Pub/Subパターンでの状態管理
fuqda
4
3.5k
RubyでPrettierを使ってみた
fuqda
0
320
RSpecによるOpen API自動テスト
fuqda
3
1.1k
OSSへの プルリクエスト作成の手引き
fuqda
2
320
Rails Girls Tokyo 13th Sponsor LT by STORES
fuqda
0
190
黒歴史リポジトリを使ったリファクタリング勉強法のススメ
fuqda
1
1.1k
Ruby2.7の新機能で簡易版Rubyインタプリタを実装してみる
fuqda
0
560
カンファレンスで技術的お土産を 増やすために出来ることを考えた
fuqda
2
510
Other Decks in Programming
See All in Programming
AIにコードを生成するコードを作らせて、再現性を担保しよう! / Let AI generate code to ensure reproducibility
yamachu
7
6.2k
Rails産でないDBを Railsに引っ越すHACK - Omotesando.rb #110
lnit
1
150
カクヨムAndroidアプリのリブート
numeroanddev
0
280
コード書くの好きな人向けAIコーディング活用tips #orestudy
77web
3
240
事業戦略を理解してソフトウェアを設計する
masuda220
PRO
18
5.5k
Using AI Tools Around Software Development
inouehi
0
850
try-catchを使わないエラーハンドリング!? PHPでResult型の考え方を取り入れてみよう
kajitack
3
460
Haskell でアルゴリズムを抽象化する / 関数型言語で競技プログラミング
naoya
10
1.5k
2度もゼロから書き直して、やっとブラウザでぬるぬる動くAIに辿り着いた話
tomoino
0
130
MLOps Japan 勉強会 #52 - 特徴量を言語を越えて一貫して管理する, 『特徴量ドリブン』な MLOps の実現への試み
taniiicom
2
630
Devinで実践する!AIエージェントと協働する開発組織の作り方
masahiro_nishimi
6
2.8k
PT AI без купюр
v0lka
0
210
Featured
See All Featured
Navigating Team Friction
lara
186
15k
Optimising Largest Contentful Paint
csswizardry
37
3.3k
The Cost Of JavaScript in 2023
addyosmani
49
8.3k
Chrome DevTools: State of the Union 2024 - Debugging React & Beyond
addyosmani
6
670
Gamification - CAS2011
davidbonilla
81
5.3k
Thoughts on Productivity
jonyablonski
69
4.7k
Intergalactic Javascript Robots from Outer Space
tanoku
271
27k
How STYLIGHT went responsive
nonsquared
100
5.6k
Sharpening the Axe: The Primacy of Toolmaking
bcantrill
42
2.4k
Templates, Plugins, & Blocks: Oh My! Creating the theme that thinks of everything
marktimemedia
30
2.4k
Raft: Consensus for Rubyists
vanstee
138
7k
Making Projects Easy
brettharned
116
6.2k
Transcript
OSS初心者がつまづきながら OSSマナーを学んでいく話 Tama Ruby!"01 2019.07.06 @fuqda90
fuqda(ふくだ)と申します! ・Tama.rb Organizor ・Ruby/Rails歴 1.5年〜 fuqdaと申します ・Tama.rb Organizor !Ruby /
Rails"1#$
fuqda(ふくだ)と申します! ・Tama.rb Organizor ・work@STORES.jp, inc. ・Ruby/Rails歴 1.5年〜 本日のお話
(OSS初心者&経験の浅いプログラマーが) 失敗の中で OSSマナーを 学んでいく話
自分は強くないけれど... 憧れの先輩方を 真似て始めてみたら 学びが深くなったので オススメしたい
こう言う人がいる
そもそもOSSって 強い人がやるものじゃないの? 自分はそんなに強くないから...
本日の目標
経験の浅い自分の取り組みを話すことで まだ取り組んだことのない方に (自分にはまだ早いと思ってる方に) 「自分もやってみようかな」 そう思ってもらうこと ※あくまで取り組み方の一つとして聞いて頂ければ幸いです
本題
【本日のメニュー】 第1章 OSSに取り組むきっかけ 第2章 初コントリビュートしたときの話 第3章 やらかし & 気付き
第1章 OSSに取り組むきっかけ 第2章 初コントリビュートしたときの話 第3章 つまづき & 気付き
業務でRailsに当てた モンキーパッチ ※コードの詳細は割愛
この経験の中で気付いた... 出来合いのライブラリを そのまま使えば良いわけでは ないかもしれない
きっと今後も 複雑なアプリの要件に 立ち向かうたびに こうした機会は訪れるだろう...
決意する!
普段使ってる OSSにパッチを送ったり 自力で直せるようになるぞ
モンキーパッチ(社内に閉じた修正) じゃなく、OSSに プルリクエストを送ってみたい!
第1章 OSSに取り組むきっかけ 第2章 初コントリビュートしたときの話 第3章 やらかし & 気付き
無い知恵を絞る...
パッチを送るために 2つのアプローチを取ることに
1. 気になるリポジトリのコードを 片っ端から追いかける 2. 業務で困っていること(不便)を考える
1. 気になるリポジトリのコードを 片っ端から追いかける 2. 業務で困っていること(不便)を考える
ある日 Railsのリポジトリを 読んでいたら...
軽微だがパッチを送れそうな箇所を発見
直す対象はわかったが... OSSって どうプルリク送れば良いのか 分からない
いろんな先輩 エンジニアの方々に 質問してみた
自分の中で目から鱗だった回答
Q. プルリクでメンテナーに分からないことを 質問してもいいものなんですか? A. 仕事と同じで長く関わっている人の方が仕様に 詳しいのは当たり前! 疑問に思ったことは質問しても大丈夫だよ!!
その後も疑問点を相談しつつ 拙い英語でプルリクを出した
この時のPRを投げる手順は以前Qiitaに 投稿したので気になる方はそちらも 参考までに...
OSS初プルリクだったもののRails にマージしてもらえた
第1章 OSSに取り組むきっかけ 第2章 初コントリビュートしたときの話 第3章 やらかし & 気付き
今度は業務にも活きる パッチを送りたい
1. 気になるリポジトリのコードを片っ端から追いかける 2. 業務で困っていること(不便)を考える
そういえば、 あれ...直したいかも
None
mongoid (gem) MongoDBをActiveRecord的に操作する ためのODM(Object Documen Mapper)
不便に思うところを 直してみた
実際にやったこと
mongoidのデフォルトでは rails g model で生成されるクラスに created_at と updated_at が付与されないので デフォルトで付与するようにした
⏰ ※これをやらない場合、毎回モデル生成のたびに rails g model モデル名 --timestamp で打つことに
例の如くプルリクを送ってみた
この時の私は自分が 最低の修正をしてしまったこと に気付いていなかった
その後プルリクに コメントが付く...
雑な訳) ActiveRecordはこのオプションを サポートしている。ActiveRecordに合わせるため、 このPRはリジェクトされるべきだ!!
最低の修正の中身
オプション(--timestampオプション) そのものを消した(最低) (このオプションそもそもいらなくない?と思ったので...)
その後、懸命な消火活動は続く
しかし、クローズしようぜ的な コメントがつき万事休す
その直後、この件について Twitterで呟いた際に 貴重なアドバイスが✨
None
None
このとき、 大切なことに気付く...
OSSにパッチを送る 行為(修正内容&PR内での振る舞い)にも マナーがあるということを
今回の場合だと mongoidのオプション そのものを消すのではなく それを生かした上で期待する修正をしてあ げるのがマナーだとわかった
@kamipoさんの アドバイスに則って 修正し直してみた
mongoidが目指す方向性に寄せて ActiveRecordのやり方に 準拠して直すようにした
None
色々やらかしましたが... なんとかmergeされることに✨
このプルリクエストは その後に修正内容に 関してマナーを意識する大 切な気付きになった
まとめ
年数は関係ない! 取り組んだ方の経験に応じた学びが きっとある!興味ある方は是非 → 当該リポジトリの思想に触れる事による学び → 自分が使っているOSSの中身を知る学び
OSSは公共物なので 使わせてもらえることを 当たり前にしない メンテナーに感謝しつつ (マナーある)PRが送れると最高かと✨
それでも一人で やるのは不安... そんな方は!
コミュニティにおいでよ!
Tama.rbでもOSSコードリーディング部 という活動をしており、 コミュニティでOSSのコードに触れる 機会を不定期で設けています (あわよくばパッチも送ったり!)
まだOSSに 取り組んだことのない方も! 是非コミュニティと一緒にやってい きましょう!!
ご静聴 ありがとう ございました!