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

fiix php8 depreciation notice #61

Closed
wants to merge 1 commit into from
Closed

fiix php8 depreciation notice #61

wants to merge 1 commit into from

Conversation

detain
Copy link

@detain detain commented Feb 15, 2024

tests need to be updated

@yadaiio
Copy link

yadaiio commented Mar 15, 2024

Hey @detain, thanks for opening this pull request.

I'm a bit unclear about the deprecation notices you mentioned and how your changes will help. It would be really helpful for me and other reviewers to see the specific error messages and understand better what you're trying to improve.

Just a heads up, the tests are currently not running green. Your changes are not the cause of this issue, as described in clue/reactphp-soap#58, because the website we are using for our test suite is currently down. Additionally, after talking with @clue and @SimonFrings about this, we have decided to address this in the future once we have identified the best approach for this. We are currently focusing on other priorities such as the ongoing development for Framework X and ReactPHP v3, so I will keep you updated once we will revisit this topic.

If you need this feature urgently, you can also reach out to us or help us prioritizing by becoming a sponsor. ❤️ Thank you!

For now I think it's best closing this until we have resolved #58 and add PHP 8 support afterwards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants