9PHP開発環境のOSS(Laradock)に初めてのプルリクを出してマージされた話PHPLaravelxhprofパフォーマンスLaraDockこんにちは。yamotukiです。普段はM&Aクラウドという八丁堀のベンチャー企業でエンジニアやってます。https://github.com/laradock/laradock/pull/2342初めてマージされた喜び出したPRも大したことないですが、私個人の心理的には大きな一歩だったので、このアドベントカレンダーのお題に乗って記録しておきたいと思います。Docker.という公式の説明を引用しますが、PHP開発環境だったら大体なんでも立ち上がる便利ツールだと解釈してます。一部では批判(※1)もありますが、お手軽に種々のPHP環境の開発環境を立ち上げたり、Dockerファイルを書くのに参考にするにはやっぱり便利なOSSです。入りすぎて初心者にはトラブルシューティングしずらいとか、立ち上げが遅いとか新規プロダクトを立ち上げて1年半ほどの弊社でも、直近では自前のDockerファイルを書く方式に移行しましたが、1年ちょっとくらいお世話になりました。OSSにコミットしたいなあ昔から「OSSにコミットしたい」という気持ちがありました。?というのを改めて考えてみました。うまく動かないそれできっかけをぼんやり探していました。tidewaysというPHPのプロファイラ(コードの性能測定)ツールを使ってみることにしたのですが、導入に多少ハマりました。tidewaysは昔PHP5系の時に有名だったが開発が止まってしまったxhprofのフォークされてPHP7対応されたものです。フラグ立てるだけで割とすぐにインストール出来ることはわかったのですが、プロファイラ開始・終了の関数がうまく呼び出せない問題がありました。tideways_enable()にすると動くというところ。自分なら直せる?ぼちぼち調べてみるとインストールされるtidewaysのバージョンの問題であることがわかりました。https://qiita.com/yamotuki/items/d8112cc7dec2c04130b4問題の理由も明快で、直すのもそこまで難しくなさそうでした。心理ハードルを超えるだけ。いうわけで初めてのOSSへのPRをチャレンジしてみることにしました。PRテンプレートの遊び心を感じる実際にPRを作って出してみると、OSSってこういう感じなのか、というのが面白かったので共有しておきます。以下はPRのテンプレートに書いてあるチェックリストです。どのガイドを読めばいいか明快に書いていて、ドキュメントもアップデートしてねと忘れそうなのを書いてある。三つ目のチェックボックスがとてもいいですね。マージされた喜びというわけでPR出していたのですが、2週間ほど放置されていました。ある程度枯れた技術になりつつあり、コミュニティはあんまり活発じゃないみたいですね。インフラ系なので動作確認が多少手間になるので確認が面倒というのもありそう。出していたPRは数分でコメントがついて、PRがイケてなかったので数日でクローズされました。勢いの差を感じる)結果として、しばらく放っておいたら特に議論もなくマージされて、自分のコードが世にでることになりました。その関連ライブラリのレポジトリにコミットしたい!一番使っているフレームワークでお世話になっていますし、コミュニティは活発。自分がPHPを実用レベルで読み書きできるようになってきたという証になると思ってます。requestStockLike11@yamotukiFollow株式会社M&AクラウドM&Aクラウドは「テクノロジーの力で、M&Aに流通革命を」をミッションにM&Aプラットフォーム「M&Aクラウド」を開発運営するスタートアップです。upLoginユーザーは見つかりませんでした。