Skip to content
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

Fix various warnings on PHP 8.2 / PHPUnit 9 #340

Merged
merged 9 commits into from
Mar 22, 2024
Merged

Fix various warnings on PHP 8.2 / PHPUnit 9 #340

merged 9 commits into from
Mar 22, 2024

Conversation

totten
Copy link
Owner

@totten totten commented Mar 12, 2024

No description provided.

totten added 4 commits March 12, 2024 15:16
Deprecated: Creation of dynamic property
CRM\CivixBundle\Builder\Module::$templateEngine is deprecated in
/home/totten/src/civix/src/CRM/CivixBundle/Builder/Module.php on line 11
Deprecated: Creation of dynamic property CRM\CivixBundle\Builder\Collection::$builders is deprecated in /home/totten/src/civix/src/CRM/CivixBundle/Builder/Collection.php on line 12
Creation of dynamic property CRM\CivixBundle\Builder\License::$license is deprecated
totten added 5 commits March 12, 2024 18:08
When running the full suite on the command-line, these tests hang because
they mistakenly execute subcommands with interactive mode. They proceed
if you press "Enter", but that's silly.
@totten totten changed the title Fix various warnings on PHP 8.2 Fix various warnings on PHP 8.2 / PHPUnit 9 Mar 13, 2024
@colemanw colemanw mentioned this pull request Mar 21, 2024
@totten
Copy link
Owner Author

totten commented Mar 22, 2024

This is passing in most permutations. The remaining failures are probably a consequence of incompatibilities between [email protected] and [email protected]. It's annoying to show formally. But I think this is better in than out.

@totten totten merged commit b912bae into master Mar 22, 2024
1 check failed
@totten totten deleted the master-compat branch March 22, 2024 07:49
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.

1 participant