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] added php84 #334

Closed

Conversation

connorhu
Copy link
Collaborator

Lets see this.

@connorhu connorhu force-pushed the feature/php84-gha-testing branch from da73ff2 to f65f6ae Compare March 12, 2024 07:48
@connorhu connorhu marked this pull request as ready for review March 12, 2024 07:50
@thePanz
Copy link
Member

thePanz commented Mar 12, 2024

php 8.4 is planned for December 2024 😅 I had to check! :)
It might be too early to have it now?

Would be cool to mark php 8.4 pipeline as "allowed to fail" on the CI, wdyt?

@connorhu
Copy link
Collaborator Author

php 8.4 is planned for December 2024 😅 I had to check! :) It might be too early to have it now?

Would be cool to mark php 8.4 pipeline as "allowed to fail" on the CI, wdyt?

@thePanz Absolutely! I'll look into how to do that.

@connorhu connorhu force-pushed the feature/php84-gha-testing branch from f65f6ae to c4addcc Compare March 18, 2024 10:09
@thirsch
Copy link
Collaborator

thirsch commented Dec 23, 2024

Meanwhile, php 8.4 has landed. Maybe it's time for updating and merging this pr?

@thePanz
Copy link
Member

thePanz commented Jan 29, 2025

PR #382 is now merged, closing this one

@thePanz thePanz closed this Jan 29, 2025
@connorhu connorhu deleted the feature/php84-gha-testing branch January 29, 2025 09:49
@connorhu
Copy link
Collaborator Author

This whole thing pisses me off, but since it's open source I guess it's the way it's supposed to work.

@thePanz
Copy link
Member

thePanz commented Jan 29, 2025

@connorhu I only looked at this PR, but completely missed that you opened #335 too!
Sorry, my bad!!

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