Upgrade to Pro — share decks privately, control downloads, hide ads and more …

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

6ce2bb5c8ac10d5224fca4105eeea70f?s=47 Doarakko
November 12, 2020

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

6ce2bb5c8ac10d5224fca4105eeea70f?s=128

Doarakko

November 12, 2020
Tweet

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