PHP CI/Unitテスト時にcacheが効いてなさそう

[k-yamasaki-zakisan]

気になった箇所

GitHub Actions上でCIをされているPHPのunit testを実行時に
毎回composer installを実行していそうでした

キャッシュを入れられているので違和感を感じました
https://github.com/ysknsid25/otaku-tool/blob/master/.github/workflows/PHP-UnitTest.yml#L35

原因

デバックをしているわけではないので予想ですが
こちらのpathが間違っていることが原因ではないかと思いました
./vendor -> ./framework/vendor
https://github.com/ysknsid25/otaku-tool/blob/master/.github/workflows/PHP-UnitTest.yml#L37

workflow post cache vendorの箇所のログ
/usr/bin/tar: /home/runner/work/otaku-tool/otaku-tool/vendor: Cannot open: No such file or directory

補足

あとキャッシュヒットしたときにautoloadが走らないのでappにクラスを追加した場合にエラーになるかもしれないので
composer dump-autoload
をいれたほうがいいかもです
参考記事

FYI

ライブラリのバージョンが古いので警告が出ておりました

• actions/checkout@v2 -> actions/checkout@v3
  • https://github.com/actions/checkout#:~:text=%2D%20uses%3A-,actions/checkout%40v3,-with%3A%0A%20%20%20%20%23
• actions/cache@v1 -> actions/cache@v3
  • https://github.com/actions/cache#:~:text=primes%0A%20%20%20%20%20%20uses%3A-,actions/cache%40v3,-with%3A%0A%20%20%20%20%20%20%20%20path

スクリーンショット

毎回composer installしてそう	cache先のフォルダがないと言われている	ライブラリが古いことによる警告

[ysknsid25]

issueありがとうございます！
こちらのissue、@k-yamasaki-zakisan のほうで対応しますか？
(面倒であれば私の方で見ていきます👍)

[k-yamasaki-zakisan]

正直このissuesの対応価値は低いと思ってます
（publicなのでActionsに料金はかかってないと思いますので...）

ただせっかくの機会なので@ysknsid25 さん自身がDevOps周りの実装をする方がいいと思いました
（仕事で新規プロジェクトに当たらないとやる機会があんまりないので...）

後お節介かもしれませんがphpstanでworkflowにlinterを入れていると品質担保に役に立つかなと思いました
参考例

• https://qiita.com/tumugin/items/9ea8b5439167b5368379

[ysknsid25]

@k-yamasaki-zakisan

いえいえとんでもないです。
こういうアドバイスを外のコミュニティの方からいただきたくて、このプロジェクトを始めたというところがあるので感謝しかないです。

おっしゃるとおり、せっかくの機会なので自分で実装させてもらいます🙏
もしよければ、なにか困った時にこのissue上で相談させていただけると助かります🙇

[ysknsid25]

@k-yamasaki-zakisan

お疲れ様です。
cache周りの問題に対して修正をおこなってみました。
もしお時間があれば、修正結果についてこちらのプルリクをレビューしていただけないでしょうか？
恐れ入りますがご確認よろしくお願いいたします。

[ysknsid25]

@k-yamasaki-zakisan