Skip to content

PHP build fixes #7

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.

Sign up for GitHub

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

Merged
aik099 merged 8 commits into from
Dec 13, 2024
Merged

PHP build fixes #7

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
e80251b
PHP build fixes
aik099 Dec 13, 2024
7d5d477
Don't test on PHP 5.4 and PHP 5.5
aik099 Dec 13, 2024
68f0f4a
Correct Prophecy integration
aik099 Dec 13, 2024
a0e06e4
Test case naming issue fix
aik099 Dec 13, 2024
dfde805
PHPUnit 10 fixes
aik099 Dec 13, 2024
16ff4e4
Don't test on PHP 8.4 due to the "aura/sql" bug
aik099 Dec 13, 2024
e5461f4
Use different PHPUnit configs for different PHPUnit versions
aik099 Dec 13, 2024
9c95cb0
Scrutinizer CI fix
aik099 Dec 13, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 26 additions & 2 deletions .github/workflows/tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,30 @@ jobs:

strategy:
matrix:
php: [ '5.4', '5.5', '5.6', '7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2', '8.3', '8.4']
include:
- php: '5.6'
phpunit_config: 'phpunit.xml.dist' # PHPUnit 5.7
- php: '7.0'
phpunit_config: 'phpunit.xml.dist' # PHPUnit 6.5
- php: '7.1'
phpunit_config: 'phpunit.xml.dist' # PHPUnit 7.5
- php: '7.2'
phpunit_config: 'phpunit.xml.dist' # PHPUnit 8.5
- php: '7.3'
phpunit_config: 'phpunit9.xml.dist' # PHPUnit 9.6
- php: '7.4'
phpunit_config: 'phpunit9.xml.dist' # PHPUnit 9.6
- php: '8.0'
phpunit_config: 'phpunit9.xml.dist' # PHPUnit 9.6
- php: '8.1'
phpunit_config: 'phpunit10.xml.dist' # PHPUnit 10.5
- php: '8.2'
phpunit_config: 'phpunit10.xml.dist' # PHPUnit 10.5
- php: '8.3'
phpunit_config: 'phpunit10.xml.dist' # PHPUnit 10.5
# Don't test on PHP 8.4 due to "aura/sql" bug: https://github.com/auraphp/Aura.Sql/pull/231.
# - php: '8.4'
# phpunit_config: 'phpunit10.xml.dist' # PHPUnit 10.5
fail-fast: false

steps:
Expand All @@ -44,7 +67,7 @@ jobs:

- name: Run tests
run: |
vendor/bin/phpunit --log-junit junit.xml --coverage-clover=coverage.clover
vendor/bin/phpunit --configuration ${{ matrix.phpunit_config }} --log-junit junit.xml --coverage-clover=coverage.clover

- name: Upload coverage to Codecov
uses: codecov/codecov-action@v4
Expand All @@ -68,5 +91,6 @@ jobs:
- name: Upload Coverage to Scrutinizer CI (PHP >= 8.0)
if: "${{ matrix.php >= '8.0' }}"
run: |
rm composer.lock
composer require scrutinizer/ocular
vendor/bin/ocular code-coverage:upload --repository=g/console-helpers/db-migration --format=php-clover coverage.clover
1 change: 1 addition & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
vendor
phpunit.xml
.phpunit.result.cache
2 changes: 1 addition & 1 deletion CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ This project adheres to [Semantic Versioning](http://semver.org/).
...

### Changed
...
- Increase minimal supported version of PHP to 5.6.

### Fixed
...
Expand Down
8 changes: 5 additions & 3 deletions composer.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,14 @@
}
],
"require": {
"php": ">=5.4",
"aura/sql": "^2.5 | ^3.0"
"php": ">=5.6",
"aura/sql": "^2.5 || ^3.0 || ^4.0 || ^5.0"
},
"require-dev": {
"aik099/coding-standard": "dev-master",
"yoast/phpunit-polyfills": "^1.0"
"yoast/phpunit-polyfills": "^2.0",
"phpspec/prophecy": "^1.10",
"console-helpers/prophecy-phpunit": "^3.0"
},
"autoload": {
"psr-4": {
Expand Down
Loading
Loading