Skip to content

Added PHPUnit 12 support and fixed tests, fixed PHP8.4 related deprecation notices, formatted to match PSR-12#51

Open
elcreator wants to merge 4 commits into
vanilla:masterfrom
elcreator:master
Open

Added PHPUnit 12 support and fixed tests, fixed PHP8.4 related deprecation notices, formatted to match PSR-12#51
elcreator wants to merge 4 commits into
vanilla:masterfrom
elcreator:master

Conversation

@elcreator

@elcreator elcreator commented May 2, 2025

Copy link
Copy Markdown

@acharron-hl I saw PHP8.4 upgrade in the garden-container so upgraded it's version in this repo too.

@CLAassistant

CLAassistant commented May 2, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@elcreator

Copy link
Copy Markdown
Author

All tests passed without failing, Psalm and Code Sniffer were also successfully executed.

@Donatello-za

Copy link
Copy Markdown

Unfortunately my old php 8.1 pull request was never accepted as well. I am assuming that the project maintainer is not active on the project and just touches it every 2 years or so.

For anyone that wants to use this PHP 8.4 pull request/fork, you can install it directly from the contributor using this:

composer config repositories.garden-cli vcs https://github.com/elcreator/garden-cli
composer require vanilla/garden-cli:dev-master

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.

3 participants