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

Require PHP 8.1+ #871

Merged
merged 4 commits into from Aug 19, 2022
Merged

Require PHP 8.1+ #871

merged 4 commits into from Aug 19, 2022

Conversation

lcobucci
Copy link
Owner

This bumps all dependencies, removes dependency on mbstring, and requires PHP 8.1+.

Signed-off-by: Luís Cobucci <lcobucci@gmail.com>
Lock file operations: 1 install, 23 updates, 1 removal
  - Removing ocramius/package-versions (2.1.0)
  - Locking composer/pcre (3.0.0)
  - Upgrading composer/xdebug-handler (1.4.6 => 3.0.3)
  - Upgrading infection/abstract-testframework-adapter (0.3.1 => 0.5.0)
  - Upgrading infection/infection (0.21.5 => 0.26.6)
  - Upgrading justinrainbow/json-schema (5.2.11 => 5.2.12)
  - Upgrading nikic/php-parser (v4.13.2 => v4.14.0)
  - Upgrading ondram/ci-detector (3.5.1 => 4.1.0)
  - Upgrading psr/container (1.1.2 => 2.0.2)
  - Upgrading sanmai/pipeline (v5.2.1 => v6.1)
  - Upgrading seld/jsonlint (1.8.3 => 1.9.0)
  - Upgrading symfony/console (v5.4.2 => v5.4.11)
  - Upgrading symfony/deprecation-contracts (v2.5.0 => v2.5.2)
  - Upgrading symfony/filesystem (v5.4.0 => v5.4.11)
  - Upgrading symfony/finder (v5.4.2 => v5.4.11)
  - Upgrading symfony/polyfill-ctype (v1.24.0 => v1.26.0)
  - Upgrading symfony/polyfill-intl-grapheme (v1.24.0 => v1.26.0)
  - Upgrading symfony/polyfill-intl-normalizer (v1.24.0 => v1.26.0)
  - Upgrading symfony/polyfill-mbstring (v1.24.0 => v1.26.0)
  - Upgrading symfony/polyfill-php73 (v1.24.0 => v1.26.0)
  - Upgrading symfony/polyfill-php80 (v1.24.0 => v1.26.0)
  - Upgrading symfony/process (v5.4.2 => v5.4.11)
  - Downgrading symfony/service-contracts (v2.5.0 => v1.1.2)
  - Upgrading symfony/string (v5.4.2 => v5.4.11)
  - Upgrading webmozart/assert (1.10.0 => 1.11.0)

Signed-off-by: Luís Cobucci <lcobucci@gmail.com>
Lock file operations: 0 installs, 19 updates, 0 removals
  - Downgrading psr/container (2.0.2 => 1.1.2)
  - Upgrading doctrine/annotations (1.13.2 => 1.13.3)
  - Upgrading doctrine/instantiator (1.4.0 => 1.4.1)
  - Upgrading doctrine/lexer (1.2.2 => 1.2.3)
  - Upgrading mikey179/vfsstream (v1.6.10 => v1.6.11)
  - Upgrading myclabs/deep-copy (1.10.2 => 1.11.0)
  - Upgrading phar-io/version (3.1.0 => 3.2.1)
  - Upgrading phpbench/phpbench (1.2.3 => 1.2.6)
  - Upgrading phpdocumentor/type-resolver (1.6.0 => 1.6.1)
  - Upgrading phpstan/phpstan (1.4.2 => 1.8.2)
  - Upgrading phpstan/phpstan-phpunit (1.0.0 => 1.1.1)
  - Upgrading phpstan/phpstan-strict-rules (1.1.0 => 1.3.0)
  - Upgrading phpunit/php-code-coverage (9.2.10 => 9.2.15)
  - Upgrading phpunit/phpunit (9.5.13 => 9.5.21)
  - Upgrading sebastian/environment (5.1.3 => 5.1.4)
  - Upgrading sebastian/global-state (5.0.3 => 5.0.5)
  - Upgrading sebastian/type (2.3.4 => 3.0.0)
  - Upgrading symfony/options-resolver (v5.4.0 => v5.4.11)
  - Upgrading symfony/service-contracts (v1.1.2 => v2.5.2)

Signed-off-by: Luís Cobucci <lcobucci@gmail.com>
This makes use of new and fancy features from PHP 8.1 and (finally)
getting rid of PHP 7.x stuff.

Signed-off-by: Luís Cobucci <lcobucci@gmail.com>
@lcobucci lcobucci added this to the 5.0.0 milestone Aug 19, 2022
@lcobucci lcobucci self-assigned this Aug 19, 2022
@lcobucci lcobucci merged commit 163bad6 into 5.0.x Aug 19, 2022
@lcobucci lcobucci deleted the require-php-81 branch August 19, 2022 20:16
@marcovtwout
Copy link

marcovtwout commented Jul 26, 2023

@lcobucci Just wondering, why did you bump the minimum required PHP version to 8.1? PHP 8.0 is still supported by PHP until the end of this year. Are you using any 8.1 specific features?

*nevermind, I see in #764b028 that lots of 8.1 specific features are applied as well.

@Ocramius
Copy link
Sponsor Collaborator

@marcovtwout used features visible in this diff: we already rely on readonly, where possible :)

Meanwhile, see https://github.com/Ocramius/.github/blob/c77629415fd5a57ef23d9372075f406e7bf98403/version-support.md#dependency-upgrades

You can expect dependency ranges to always be restricted further, as time passes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants