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

PHP8.4 Support Added #38

Open
wants to merge 5 commits into
base: 2.18.x
Choose a base branch
from
Open

Conversation

cod43156
Copy link

Q A
Documentation yes/no
Bugfix yes/no
BC Break yes/no
New Feature yes/no
RFC yes/no
QA yes/no

@cod43156
Copy link
Author

@froschdesign Could you kindly proceed with merging this PR if there are no further concerns? Thank you!

@cod43156
Copy link
Author

@froschdesign, just checking in to see if you've had a chance to review this PR. Could you kindly proceed with merging it if everything looks good? Thank you!

@nishant04412
Copy link

@gsteel Can you please review and let us know if this can be merged or anything needs to be addressed ?

@froschdesign
Copy link
Member

@nishant04412
It is absolutely not necessary to ask for reviews, because nothing will be overlooked. Please remember, we are only doing this in our spare time and there is still time before the official release of version 8.4.

@cod43156
Copy link
Author

@gsteel @froschdesign we understand things are busy, and reviews happen on your own time. Whenever you can, we'd appreciate a quick look to ensure we're on the right track. No worries if it's not immediate, just whenever it fits into your schedule!

@cod43156
Copy link
Author

cod43156 commented Jan 2, 2025

Hi @gsteel @froschdesign

Just wanted to gently follow up on the review request whenever you have the time.

Thank you so much for your support!

.laminas-ci.json Outdated Show resolved Hide resolved
Signed-off-by: Raj Mohan <[email protected]>
Signed-off-by: Raj Mohan <[email protected]>
composer.json Outdated Show resolved Hide resolved
Signed-off-by: Raj Mohan <[email protected]>
@samsonasik
Copy link
Member

composer.lock needs to be regenerated

composer.json Outdated Show resolved Hide resolved
@froschdesign froschdesign added this to the 2.18.0 milestone Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

4 participants