Skip to content

maintenance update for phpunit-12 #12

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

kazweda
Copy link
Contributor

@kazweda kazweda commented May 19, 2025

#11 Update to run on the current stable version.

PHP8.3以上、phpunit-12で実行できるようにしてみました。

@kazweda
Copy link
Contributor Author

kazweda commented May 20, 2025

@twada ふと思いついて更新してみました。
エラー回避のため phpunit.xml.dist を修正しています。
不都合があればコメントお願いします。

@shishi
Copy link
Member

shishi commented May 20, 2025

phpの入ってない環境にphp8.4をインストールして実行してみました。

curl -sS https://getcomposer.org/installer | php
php composer.phar install

でinstall すると update するようにとエラーになったので php composer.phar update する必要がありました。 update後

./vendor/bin/phpunit

は成功しました。PHPから離れてひさしいので良くわからないのですが、これはreadmeに記載しておくとよさそうですかね?

@kazweda
Copy link
Contributor Author

kazweda commented May 20, 2025

@shishi 動作確認ありがとうございます。

こちらは homebrew の php 環境です。
php-8.4.3 を 8.4.7 に更新して install してみましたが update が必要な状況にはなりませんでした。
他の環境で試してみます。

@shishi
Copy link
Member

shishi commented May 21, 2025

私はUbuntu24.04に nix profile install nixpkgs#php したものですが、インストール方法はたくさんあるものですから、起こりうることならそう記載しておけばよいかなとー。

@kazweda
Copy link
Contributor Author

kazweda commented May 21, 2025

了解です。情報共有ありがとうございます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants