PHPStan を require-dev に入れる

[nanasess]

概要(Overview・Refs Issue)

方針(Policy)

実装に関する補足(Appendix)

• PHPStan を composer.json の require-dev に入れる
  • エラーの出ていない level: 1 に設定
• Variable $PrevPage might not be defined を修正
• PHPStan による静的解析の導入 #4835

開発中にローカル環境で実行するには、以下のコマンドを実行する

vendor/bin/phpstan analyze src

テスト（Test)

GitHub Actions でエラーの出ないことを確認

相談（Discussion）

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[okazy]

ありがとうございます。
動作確認して問題ありませんでした。

コードの変更も問題ありません。

[okazy]

いくつかテストが落ちていましたが、今回の変更箇所とは無関係のところでしたので問題ないかと思います。
念の為再実行しておきました。

[okazy]

落ちているテストは他のプルリクで対応中のものでしたのでマージしました。ご対応ありがとうございました。