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

Enhancement: Add support for PHP 8.0 and updated minimal php version to 7.3 #211

Closed
2 tasks done
EJTJ3 opened this issue Apr 6, 2021 · 0 comments · Fixed by #212
Closed
2 tasks done

Enhancement: Add support for PHP 8.0 and updated minimal php version to 7.3 #211

EJTJ3 opened this issue Apr 6, 2021 · 0 comments · Fixed by #212

Comments

@EJTJ3
Copy link
Contributor

EJTJ3 commented Apr 6, 2021

Due to the deprecated composer v1, and supported php versions we need to upgrade some of the dependencies for this bundle:

https://blog.packagist.com/deprecating-composer-1-support/

What should happen:

  1. Upgrade minimal PHP version to 7.3 | 8.0 (https://www.php.net/supported-versions.php)
  2. Upgrade phpro/grumphp to version 1.*
  3. Upgrade phpunit/phpunit to version 9
  4. Upgrade phpstan/phpstan to version 0.12
  5. Upgrade guzzlehttp/guzzle to version 7 (Upgrade to Laravel 8 not possible due to "old" guzzlehttp/guzzle #204)
  6. Replace jakub-onderka/php-parallel-lint with php-parallel-lint/php-parallel-lint
  7. Remove sebastian/phpcpd (deprecated)
  8. Remove sensiolabs/security-checker (deprecated)

What happens:

  1. Losing support for php7.1 & php7.2
  2. Adding support for php8.0
  3. Adding support for composer v2

SDK version and environment

  • Tested on 1.14.1
  • Sandbox
  • Production
EJTJ3 pushed a commit to EJTJ3/sdk_php that referenced this issue Apr 6, 2021
@EJTJ3 EJTJ3 changed the title Update minimal requirements for this bundle Enhancement: Add support for PHP 8.0 and updated minimal php version to 7.3 Sep 15, 2021
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 a pull request may close this issue.

1 participant