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

chore: Add tests for PHP 8.4 #2871

Merged
merged 4 commits into from
Nov 22, 2024
Merged

Conversation

Chris53897
Copy link
Contributor

No description provided.

@Chris53897 Chris53897 closed this Nov 22, 2024
@Chris53897 Chris53897 reopened this Nov 22, 2024
@Chris53897
Copy link
Contributor Author

@phansys CI is green now.
php-cs-fixer should be handled in seperate PR i guess,

@phansys
Copy link
Collaborator

phansys commented Nov 22, 2024

@phansys CI is green now.

Thanks @Chris53897.

php-cs-fixer should be handled in seperate PR i guess,

Are there issues in the remaining GHA workflows related to the resolved PHP version? Is there something we can do to fix them in order to provide a complete bump in this PR?

@Chris53897
Copy link
Contributor Author

No. I think this can be merged as it is.

@phansys
Copy link
Collaborator

phansys commented Nov 22, 2024

So, what prevent us to bump the PHP version in the remaining workflow jobs?

@Chris53897
Copy link
Contributor Author

Sorry. Now i get your point.

PHP-CS-Fixer needs to be at PHP 8.3.
I updated the others.

@phansys phansys merged commit b48127c into doctrine-extensions:main Nov 22, 2024
21 checks passed
@phansys
Copy link
Collaborator

phansys commented Nov 22, 2024

Thank you @Chris53897!

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