Skip to content

Require PHP 8.1+#871

Merged
lcobucci merged 4 commits into
5.0.xfrom
require-php-81
Aug 19, 2022
Merged

Require PHP 8.1+#871
lcobucci merged 4 commits into
5.0.xfrom
require-php-81

Conversation

@lcobucci

Copy link
Copy Markdown
Owner

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

Signed-off-by: Luís Cobucci <[email protected]>
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 <[email protected]>
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 <[email protected]>
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 <[email protected]>
@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

marcovtwout commented Jul 26, 2023

Copy link
Copy Markdown

@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
Copy Markdown
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.

3 participants