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

Test on PHP 8.0 #2268

Closed
wants to merge 1 commit into from
Closed

Test on PHP 8.0 #2268

wants to merge 1 commit into from

Conversation

driesvints
Copy link
Contributor

@driesvints driesvints commented Jul 8, 2021

This also allows the library to be built on PHP 8.

@driesvints
Copy link
Contributor Author

I can't seem to start the tests to run unless I get approval because of GitHub's new policies. Therefor I have no idea if the test suite will pass or not. After approval has been given I can look into making the tests pass.

I also can't run them locally because an incompatible PHPUnit version is loaded because of its very loose constraints: https://github.com/sebastianbergmann/phpunit/blob/4.8.36/composer.json#L23

@ajredniwja ajredniwja requested a review from SamRemis July 8, 2021 21:05
@driesvints
Copy link
Contributor Author

Going to close this one as there's much work to be done to migrate the entire test suite to a newer PHPUnit version. I unfortunately don't have the time for this.

@driesvints driesvints closed this Jul 9, 2021
@driesvints driesvints deleted the php8-tests branch July 9, 2021 12:08
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.

1 participant