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

feat: PHP 8.3 #487

Closed
wants to merge 3 commits into from
Closed

feat: PHP 8.3 #487

wants to merge 3 commits into from

Conversation

dkarlovi
Copy link
Contributor

No description provided.

@jakzal
Copy link
Owner

jakzal commented Nov 27, 2023

Ugh... this one's going to be a bit more involved.

I think I might need to rip out prophecy and replace it with PHPUnit mocks.

@dkarlovi
Copy link
Contributor Author

I got it partially working locally, but it needs to use yet unreleased packages, moving to PHPUnit mocks is probably for the best long term.

@jakzal
Copy link
Owner

jakzal commented Nov 28, 2023

Prophecy is gone.

@dkarlovi
Copy link
Contributor Author

I just got a notification on my phone saying (without context)

Prophecy is gone.

Seemed like Zeus was sending me a text. 🤣

@dkarlovi
Copy link
Contributor Author

build fails, did you mean to release 2.7.0 here?
https://github.com/jakzal/phpunit-globals/releases/tag/v3.2.0

@jakzal
Copy link
Owner

jakzal commented Nov 28, 2023

No. Let's drop PHP 8.0 here if it's causing problems.

@dkarlovi
Copy link
Contributor Author

Makes sense, 8.0 is EOL.

This was referenced Nov 28, 2023
@dkarlovi
Copy link
Contributor Author

Replaced by #493

@dkarlovi dkarlovi closed this Nov 29, 2023
@jakzal
Copy link
Owner

jakzal commented Nov 29, 2023

@dkarlovi I had to create a new PR so that builds could use secrets.

@dkarlovi
Copy link
Contributor Author

No problem! 🍻

@dkarlovi dkarlovi deleted the feat/8.3 branch November 29, 2023 11:31
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.

2 participants