バルサ 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 が使っているバージョン 最新バージョン

17. 17

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