Skip to content

Commit

Permalink
Drop PHP 7.2 and 7.3
Browse files Browse the repository at this point in the history
  • Loading branch information
ondrejmirtes authored and kukulich committed Nov 7, 2024
1 parent 0dfa3c0 commit 88602f9
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 8 deletions.
8 changes: 2 additions & 6 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,6 @@ jobs:
dependencies:
- "highest"
php-version:
- "7.2"
- "7.3"
- "7.4"
- "8.0"
- "8.1"
Expand All @@ -45,7 +43,7 @@ jobs:
ini-values: error_reporting=-1, display_errors=On, zend.assertions=1

- name: "Reset composer platform"
if: matrix.php-version == '7.2' || matrix.php-version == '7.3' || matrix.php-version == '7.4'
if: matrix.php-version == '7.4'
run: "composer config --unset platform"

- name: "Install dependencies"
Expand Down Expand Up @@ -135,8 +133,6 @@ jobs:
- "lowest"
- "highest"
php-version:
- "7.2"
- "7.3"
- "7.4"
- "8.0"
- "8.1"
Expand All @@ -159,7 +155,7 @@ jobs:
ini-values: error_reporting=-1, display_errors=On, zend.assertions=1

- name: "Reset composer platform"
if: matrix.php-version == '7.2' || matrix.php-version == '7.3' || matrix.php-version == '7.4' || matrix.php-version == '8.0'
if: matrix.php-version == '7.4' || matrix.php-version == '8.0'
run: "composer config --unset platform"

- name: "Install dependencies"
Expand Down
4 changes: 2 additions & 2 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
}
},
"require": {
"php": "^7.2 || ^8.0",
"php": "^7.4 || ^8.0",
"dealerdirect/phpcodesniffer-composer-installer": "^0.6.2 || ^0.7 || ^1.0",
"phpstan/phpdoc-parser": "^2.0",
"squizlabs/php_codesniffer": "^3.9.0"
Expand All @@ -28,7 +28,7 @@
"phpstan/phpstan-deprecation-rules": "1.1.4",
"phpstan/phpstan-phpunit": "1.3.16",
"phpstan/phpstan-strict-rules": "1.5.2",
"phpunit/phpunit": "8.5.21|9.6.8|10.5.11"
"phpunit/phpunit": "9.6.8|10.5.11"
},
"autoload": {
"psr-4": {
Expand Down

1 comment on commit 88602f9

@bkdotcom
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does the codebase contain code, syntax, or dependancy that requires a minimum of Php 7.4?

Please sign in to comment.