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

PHP 8 support #65

Merged
merged 9 commits into from Jul 6, 2020
Merged

PHP 8 support #65

merged 9 commits into from Jul 6, 2020

Conversation

GrahamCampbell
Copy link
Contributor

@GrahamCampbell GrahamCampbell commented Jun 8, 2020

TODO:

  • Update composer.json
  • Fix source code
    • Fix FactoryParameter for PHP 7.1-8.0
    • Fix FactoryGenerator for newer versions of the finder
    • Corrected generation to be platform independent (always generate unix style line endings)
    • Regenerate functions file
  • Update tests and CI
    • Test on PHP 8
    • Fix coverage reporting Remove broken coverage reporting

EDIT 1: I will use GitHub Actions to run the PHP 7+ builds, and Travis for PHP 5 and HHVM 3.18.

EDIT 2: This repo is actually not enabled on scrutinizer or coveralls, and coverage reporting has been broken for years, so I have removed it.


Related to mockery/mockery#1072, sebastianbergmann/phpunit#4325.

@GrahamCampbell GrahamCampbell marked this pull request as ready for review July 5, 2020 12:00
@GrahamCampbell
Copy link
Contributor Author

GrahamCampbell commented Jul 5, 2020

Note that GitHub actions didn't run because the config is only present in my fork, but it will run along side travis once this is merged. The associated (passing) actions run from my fork can be found here: https://github.com/GrahamCampbell/hamcrest-php/actions/runs/158263545.

@GrahamCampbell
Copy link
Contributor Author

@davedevelopment This is ready for review now. :)

Copy link

@Ayesh Ayesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 🙏🏿

@davedevelopment davedevelopment merged commit 459c788 into hamcrest:master Jul 6, 2020
@davedevelopment
Copy link
Member

Thank you again Graham 🙌

@GrahamCampbell GrahamCampbell deleted the patch-1 branch July 6, 2020 18:31
@GrahamCampbell
Copy link
Contributor Author

@davedevelopment No problem at all. :) Let me know if there's anything else outstanding that you'd like help with before you're happy to tag the next release (2.0.1 I assume). 🚢

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.

None yet

3 participants