バルサ CL 初戦大勝のその後

Transcript

1. バルサ CL 初戦大勝のその後 @Doarakko
2. ・2019年12月入社（2社目） ・エンジニア歴1.5年 　- 仕事: Go, PHP（たまに） 　- 趣味: Python, C++ ・AtCoder 緑 ・海外サッカーをよく見ます 自己紹介 2
3. CL 初戦前にヘタフェに完封負け 3
4. 【朗報】バルセロナ大量5得点、圧倒的勝利！ 4
5. Gin にコミットしたい >>>>> 眠気 5
6. コミットしました 6 パッケージのバージョン上げただけだよ
7. ・Go の Web フレームワーク ・外資就活ドットコムのバックエンドで使用 ・OSS（Open Source Software） Gin とは 7
8. ・ブログを書く ・勉強会を開催 ・issue を立てる ・ドキュメントの追加・修正・翻訳 ・機能追加 ・バグ修正 OSS に貢献するには 8 他にもあるかも
9. ・ブログを書く ・勉強会を開催 ・issue を立てる ・ドキュメントの追加・修正・翻訳 ・機能追加 ・不具合・バグの修正 OSS にコミットするには 9 他にもあるかも
10. issue 探し 10 OSS によっては「good first issue」というラベルをつけてくれています！
11. Undefined validation function 'required_if' on field 'Field' 11 issue template に合わせて書いてくれているのでローカルでの再現や対応がしやすかった
12. Gin の binding タグとは 12 Gin の内部では validator パッケージを使っているので挙動は同じ Gin binding タグ validator の validate タグ
13. binding タグで「required_if」を指定すると定義されてませんよエラー 13
14. ・ローカルで再現するか 　→ 再現 ・「required_if」以外を指定するとどうなかるか 　→ 同様のエラーになるものが一部あり ・validator パッケージの issue に報告が上がっていないか 　→ 特になし ・（Gin のバージョンを最新にしても再現するか） 一つずつ確認していく 14
15. Gin は内部で validator パッケージを使用している 15
16. validator パッケージのバージョンを見てみると... 16 Gin が使っているバージョン 最新バージョン
18. required_if がリリースされたバージョンを調べてみる 18 ソースから PR を特定して Releases からバージョンを確認
19. Gin が使っているバージョンでは「required_if」が追加されていない 19 validator パッケージのバージョンあげるぞ
20. PR を上げるまでの手順 20
21. ・README.md を読む ・CODE_OF_CONDUCT.md を読む ・CONTRIBUTING.md を読む ・コメントがついている issue や PR を見る（雰囲気を知るため） OSS に PR を上げる前に⚠ 21 ルールが厳格な OSS もあるので必ず読む
22. ① リポジトリを Fork してローカルに落とす 22 本家に直で Push する権限がないため
23. ② go get -u github.com/go-playground/validator/v10 23 動作確認・テストコードの実行も忘れずに！
24. ③ PR 作成 24 マージされた他の方の PR を見て雰囲気を掴む
25. 05:53 25
26. 12:26 26
27. マージされました 27
28. ・OSS 活動怖くない ・小さな修正でも多くの人のためになる ・issue をよく見てみると意外とコミットチャンスあるかも まとめると... 28
29. バルサ CL 初戦大勝のその後 ＠Doarakko