Skip to content

feat: Upgraded to PHP 8, dropped PHP <7.4, upgraded dependencies #97

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

Conversation

Ocramius
Copy link
Contributor

  • Updated tests to be phpunit 9.5+-compliant
  • upgraded gitlabphp/client, which came with some BC breaks
  • applied CS fixes from upgraded laminas/coding-standard

Fixes #96

@Ocramius
Copy link
Contributor Author

Likely need to run with laxer constraints or smaller dependency ranges here.

WDYT about 7.4+ only, @weierophinney?

@Ocramius
Copy link
Contributor Author

Picking this up again now: 7.3 obviously a problem :D

@Ocramius Ocramius changed the title feat: Upgraded to PHP 8, dropped PHP <7.3, upgraded dependencies feat: Upgraded to PHP 8, dropped PHP <7.4, upgraded dependencies Jan 20, 2021
@weierophinney weierophinney added the enhancement New feature or request label Jan 20, 2021
@weierophinney weierophinney added this to the 2.11.0 milestone Jan 20, 2021
Ocramius and others added 5 commits January 20, 2021 16:23
 * Updated tests to be phpunit 9.5+-compliant
 * upgraded `gitlabphp/client`, which came with some BC breaks
 * applied CS fixes from upgraded `laminas/coding-standard`
Dependency requirements are explicitly stated by `composer` at installation
time, should the current installation target environment not be compatible.
- CaptainHook can work either with a globally installed PHAR, or with a local vendor binary.
  This adds it as a local vendor binary to ensure it is present on all systems.
- Updates the PHPUnit configuration to latest schema.

Signed-off-by: Matthew Weier O'Phinney <[email protected]>
Signed-off-by: Matthew Weier O'Phinney <[email protected]>
@weierophinney weierophinney force-pushed the feature/#96-update-dependencies-and-allow-php-8 branch from 8632c3b to 5284731 Compare January 20, 2021 22:26
@weierophinney weierophinney changed the base branch from 2.10.x to 2.11.x January 20, 2021 22:26
@weierophinney weierophinney merged commit 2afe7d8 into phly:2.11.x Jan 20, 2021
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
None yet
Development

Successfully merging this pull request may close these issues.

php:~8.0.0 support
2 participants