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

Add PHP 8 Support #8

Merged
merged 2 commits into from Oct 9, 2020

Conversation

weierophinney
Copy link
Contributor

@weierophinney weierophinney commented Oct 9, 2020

  • Added ~8.0.0 to the PHP constraint
  • Bumped minimum supported PHPUnit version to 9.3
  • Added PHP 8 jobs to Travis configuration
  • Updated unit tests to work under PHPUnit 9.3
    • Replaced Prophecy mocking with PHPUnit native mocking
  • Updates to laminas-coding-standard 2.1
    • Applies rules to codebase

Fixes #5

- Added `~8.0.0` to the PHP constraint
- Bumped minimum supported PHPUnit version to 9.3
- Added PHP 8 jobs to Travis configuration
- Updated unit tests to work under PHPUnit 9.3
  - Replaced Prophecy mocking with PHPUnit native mocking

Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
- Updates to laminas-coding-standard 2.1
- Applies rules to codebase

Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
@weierophinney weierophinney added this to To do in PHP 8.0 via automation Oct 9, 2020
@weierophinney weierophinney added this to the 1.5.0 milestone Oct 9, 2020
@weierophinney weierophinney added the Enhancement New feature or request label Oct 9, 2020
@weierophinney weierophinney moved this from To do to In progress in PHP 8.0 Oct 9, 2020
@weierophinney
Copy link
Contributor Author

Unclear why phpcs is failing to find the laminas-coding-standard. I can run the exact same commands as Travis locally, using PHP 7.4.11, and everything works fine. I'll see if it resolves with the next patch I do for Psalm support.

weierophinney added a commit that referenced this pull request Oct 9, 2020
Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
@weierophinney weierophinney merged commit d3f2748 into mezzio:1.5.x Oct 9, 2020
PHP 8.0 automation moved this from In progress to Done Oct 9, 2020
@weierophinney weierophinney deleted the feature/php-8-support branch October 9, 2020 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
No open projects
PHP 8.0
  
Done
Development

Successfully merging this pull request may close these issues.

PHP 8.0 support
1 participant