phpcon沖縄2023_初めてのOSSコントリビュートに挑戦したら、既に該当機能が存在していた件について

Transcript

1. 初めての OSSコントリビュート に挑戦したら、
 既に 該当機能 が 存在 していた件について
 
 2023/09/16
 
 (株)インフィニットループ　
 サーバーサイドエンジニア　kerokero

2. 今回の発表の対象者
 ・ OSSコントリビュートに興味がある初心者
 

3. 今回の発表の対象者
 ・ OSSコントリビュートに興味がある初心者
 ・ PHPやその周辺技術に貢献したい人
 

4. 今回の発表の対象者
 ・ OSSコントリビュートに興味がある初心者
 ・ PHPやその周辺技術に貢献したい人
 ・ laravel/frameworkにPRを送ってみたい人

5. 今回の発表の対象者
 ・ OSSコントリビュートに興味がある初心者
 ・ PHPやその周辺技術に貢献したい人
 ・ laravel/frameworkにPRを送ってみたい人
 < 3つです

6. Agenda
 1.　軽い自己紹介
 
 2.　laravel/framework にPRを出そうとした経緯
 
 3.　実際にPRを出してみた
 
 4.　それって〇〇と何が違うんだい？
 
 5.　反省点・良かった点・感想

7. Agenda
 1.　軽い自己紹介　　　　　　　　　　
 
 2.　laravel/framework にPRを出そうとした経緯
 
 3.　実際にPRを出してみた
 
 4.　それって〇〇と何が違うんだい？
 
 5.　反省点・良かった点・感想

8. どうも、kerokero です。

9. 22卒　自称青魔道士系エンジニア 
 （株）インフィニットループ 
 kerokero ・北海道の 札幌 生まれ、札幌 育ち
 ・水産→数学(非情報系)→2022年4月にIL入社 　 ・最近の趣味は 絵を描く ことと ストリートファイター6 です
 ・担当領域は サーバーサイドのアプリ・インフラ実装(PHP, AWSなど) です 

10. Agenda
 1.　軽い自己紹介　　　　　　　　　　
 
 2.　laravel/framework にPRを出そうとした経緯
 
 3.　実際にPRを出してみた
 
 4.　それって〇〇と何が違うんだい？
 
 5.　反省点・良かった点・感想

11. というわけでいきなり本題ですが……

12. というわけでいきなり本題ですが……
 Laravel の アップグレード をしようとしました
 

13. Laravel の アップグレード をしようとしました
 
 というわけでいきなり本題ですが……
 ・チームで使っているLaravelのバージョンを9から10に移行することに
 
 ・この作業は kerokero が担当することになった
14. すると……

15. psalm/plugin-laravel もアップデートされてた‼
 すると……
 ・Psalmプラグインの更新により、コードの静的解析の結果が変わった

16. psalm/plugin-laravel もアップデートされてた‼
 すると……
 ・Psalmプラグインの更新により、コードの静的解析の結果が変わった
 < なんとかして、大幅に実装を変えることなく静的解析が通るようにしたい...... 

17. 具体的には……

18. 具体的には……
 Illuminate/Support/Collection の get メソッドを使っている部分 が大量に通らなくなってしまった ‼
 

19. 具体的には……
 Illuminate/Support/Collection の get メソッドを使っている部分 が大量に通らなくなってしまった ‼
 
 ERROR: PossiblyNullPropertyFetch - app/Services/Queries/Weapon/WeaponQuery.php:87:19 
 - Cannot get property on possibly null variable $masterWeapon of type mixed|null (see https://psalm.dev/082)
 ? $masterWeapon->id
 $masterWeapon = $mstWeaponsByMaterial Count->get($masterByMaterial->materialCount); 
 psalm/plugin-laravel (v2.8.0) 

20. - Cannot get property on possibly null variable $masterWeapon of type mixed|null (see https://psalm.dev/082)
 nullが入る可能性があるよって言われてる

21. - Cannot get property on possibly null variable $masterWeapon of type mixed|null (see https://psalm.dev/082)
 nullが入る可能性があるよって言われてる
 null って mixed に含まれてるんじゃないの？　 >
22. そもそもアプリの仕様上 null が出ることはない......

23. そもそもアプリの仕様上 null が出ることはない......
 null なら 例外 を吐けばよさそう！
 < やったー！

24. first メソッドには、 
 firstOrFail メソッドがある……

25. first メソッドには、 
 firstOrFail メソッドがある……
 一方、get メソッドには、 
 getOrFail メソッドがない……‼
 まじかよ...... >
26. というわけで......

27. というわけで......
 getOrFail メソッド追加すればいいんじゃない？
28. Agenda
 1.　軽い自己紹介　　　　　　　　　　
 
 2.　laravel/framework にPRを出そうとした経緯
 
 3.　実際にPRを出してみた　　　　　　 
 
 4.　それって〇〇と何が違うんだい？
 
 5.　反省点・良かった点・感想

29. まずは laravel/framework への PR の出し方を調べる

30. まずは laravel/framework への PR の出し方を調べる
 ・OSS によって手順やルールが異なるので注意が必要
 
 ・今回は Laravel が指定している方法を確認する必要がある

31. まずは laravel/framework への PR の出し方を調べる
 ・OSS によって手順やルールが異なるので注意が必要
 
 ・今回は Laravel が指定している方法を確認する必要がある
 AI いらすとや　すごい　 >
32. Laravel の場合は公式が Contribution Guide というものを出してる
 ・https://laravel.com/docs/10.x/contributions
 

33. Laravel の場合は公式が Contribution Guide というものを出してる
 ・https://laravel.com/docs/10.x/contributions
 
 ・ブランチの切り方や、コーディングスタイルの指定などがある

34. Laravel の場合は公式が Contribution Guide というものを出してる
 ・https://laravel.com/docs/10.x/contributions
 
 ・ブランチの切り方や、コーディングスタイルの指定などがある
 
 ・resources/css や resources/js など、他のコンパイル済みファイルに影響を与え るものに関しては、メンテナが別途生成しコミットする 　　　　などなど
35. 具体的な作業としては

36. 具体的な作業としては
 ・laravel/frameworkをforkして、手元にcloneしてくるところから 始める
 

37. 具体的な作業としては
 
 ・laravel/frameworkをforkして、手元にcloneしてくるところから 始める
 
 ・手元で変更箇所を実装したら、Conventional Commitsというや り方に則ってコミットを積んであげる
 
 fix: koko kesuto ugokanai >
38. Conventional Commits とは

39. Conventional Commits とは
 ・https://www.conventionalcommits.org/en/v1.0.0/
 
 ・Conventional Commits というのは、コミットメッセージ上の規約のこと 
 
 
 
 

40. Conventional Commits とは
 ・https://www.conventionalcommits.org/en/v1.0.0/
 
 ・Conventional Commits というのは、コミットメッセージ上の規約のこと 
 
 ・feat(lang): add Polish language みたいなやつ
 
 
 

41. Conventional Commits とは
 ・https://www.conventionalcommits.org/en/v1.0.0/
 
 ・Conventional Commits というのは、コミットメッセージ上の規約のこと 
 
 ・feat(lang): add Polish language みたいなやつ
 
 ・セマンティック バージョニング に対応しており、自動化しやすい 
 
 
 

42. Conventional Commits とは
 ・https://www.conventionalcommits.org/en/v1.0.0/
 
 ・Conventional Commits というのは、コミットメッセージ上の規約のこと 
 
 ・feat(lang): add Polish language みたいなやつ
 
 ・セマンティック バージョニング に対応しており、自動化しやすい 
 ・多くのOSSではConventional Commitsのやり方を取り入れている 
 
 

43. 実際にコミットした内容について英語でメッセージを書いていきます
 

44. 実際にコミットした内容について英語でメッセージを書いていきます
 
 ・日本語で書いて deepl を使い翻訳したものを確認したものを使いました
 
 

45. 実際にコミットした内容について英語でメッセージを書いていきます
 
 ・日本語で書いて deepl を使い翻訳したものを確認したものを使いました
 
 

46. 実際にコミットした内容について英語でメッセージを書いていきます
 
 ・日本語で書いて deepl を使い翻訳したものを確認したものを使いました
 
 
 他の人のＰＲなども参考にしたよ > 

47. というわけでいよいよPR提出していくのですが……

48. というわけでいよいよPR提出していくのですが……
 結論から行くと　うまくいきませんでした

49. Agenda
 1.　軽い自己紹介　　　　　　　　　　
 
 2.　laravel/framework にPRを出そうとした経緯
 
 3.　実際にPRを出してみた
 
 4.　それって〇〇と何が違うんだい？　　　
 
 5.　反省点・良かった点・感想

50. 実際にPRをだすと、GitHub上でCIが動き出します
 
 ・各種指定したバージョンでのテストを通過することが必要だった
51. 実際にPRをだすと、GitHub上でCIが動き出します
 
 ・各種指定したバージョンでのテストを通過することが必要だった ・なかなかテストが通らなかったので、手元で修正してコミットを積んでた
52. 実際にPRをだすと、GitHub上でCIが動き出します
 
 ・各種指定したバージョンでのテストを通過することが必要だった ・なかなかテストが通らなかったので、手元で修正してコミットを積んでた ・テストが通ったら squash してコミットをまとめようと思ってた
53. 実際にPRをだすと、GitHub上でCIが動き出します
 
 ・各種指定したバージョンでのテストを通過することが必要だった ・なかなかテストが通らなかったので、手元で修正してコミットを積んでた ・テストが通ったら squash してコミットをまとめようと思ってた
54. すると、まもなくしてレビューコメントをいただきました‼
 

55. すると、まもなくしてレビューコメントをいただきました‼
 
 それって sole メソッドと何が違うんだい？

56. すると、まもなくしてレビューコメントをいただきました‼
 
 それって sole メソッドと何が違うんだい？
 sole ...... ？ >
57. というわけで、sole メソッドについて調べてみた‼

58. というわけで、sole メソッドについて調べてみた‼

59. というわけで、sole メソッドについて調べてみた‼

60. というわけで、sole メソッドについて調べてみた‼
 < たしかに、既にあったわ…… 

61. Agenda
 1.　軽い自己紹介　　　　　　　　　　
 
 2.　laravel/framework にPRを出そうとした経緯
 
 3.　実際にPRを出してみた
 
 4.　それって〇〇と何が違うんだい？
 
 5.　反省点・良かった点・感想　　　　

62. 今回の反省点・分かった点・感想
63. 今回の反省点・分かった点・感想 ・事前に下調べを入念に重ねてから PR を出すべきだった
 
 
 

64. 今回の反省点・分かった点・感想 ・事前に下調べを入念に重ねてから PR を出すべきだった
 
 ・内容はさておき、PRを出すこと自体はそれほど難しくないOSSも存 在する
 
 
 

65. 今回の反省点・分かった点・感想 ・事前に下調べを入念に重ねてから PR を出すべきだった
 
 ・内容はさておき、PRを出すこと自体はそれほど難しくないOSSも存 在する
 
 ・Laravelに限らずphp関連のOSSのコードを理解しようと思うと、どうして もある程度php-srcの理解が必要になる
 

66. 今回の反省点・分かった点・感想 ・事前に下調べを入念に重ねてから PR を出すべきだった
 
 ・内容はさておき、PRを出すこと自体はそれほど難しくないOSSも存 在する
 
 ・Laravelに限らずphp関連のOSSのコードを理解しようと思うと、どうして もある程度php-srcの理解が必要になる
 
 今回はissueや過去のPRも調べたつもりだったけど、そもそも既にある機能だというところが盲点だった
67. ご清聴ありがとうございました!!