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

ci: Add tests for PHP 8.4 #390

Open
wants to merge 1 commit into
base: 4.x
Choose a base branch
from
Open

Conversation

Chris53897
Copy link
Contributor

No description provided.

@alamirault
Copy link

vimeo/psalm#10928 (comment) will be available "the next week."

(I can't unapprove now)

@egulias
Copy link
Owner

egulias commented Dec 26, 2024

Hi @Chris53897 thanks for your PR. I have fixed the checks so please update with the latest code to have the checks pass.
Also, I believe that many deps are still not there for PHP8.4 but please double check.
Thanks!

@Chris53897 Chris53897 closed this Dec 27, 2024
@Chris53897 Chris53897 reopened this Dec 27, 2024
@egulias
Copy link
Owner

egulias commented Dec 27, 2024

Looks like at least psalm version needs to be bumped up.

@Chris53897
Copy link
Contributor Author

i rebased now. Yes, we still ned to wait for vimeo

@egulias
Copy link
Owner

egulias commented Dec 31, 2024

Thanks! Will wait for them then.

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