Skip to content

Commit e8a1cfc

Browse files
authored
Merge pull request #1210 from maartenderie/master
RdKafka; Replace composer-modifying for testing with --ignore-platform-req argument
2 parents 65498e3 + 293817d commit e8a1cfc

File tree

2 files changed

+4
-11
lines changed

2 files changed

+4
-11
lines changed

pkg/rdkafka/.github/workflows/ci.yml

+4-2
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,10 @@ jobs:
2222
php-version: ${{ matrix.php }}
2323
coverage: none
2424

25-
- run: php Tests/fix_composer_json.php
26-
2725
- uses: "ramsey/composer-install@v1"
26+
with: # ext-rdkafka not needed for tests, and a pain to install on CI;
27+
composer-options: "--ignore-platform-req=ext-rdkafka"
28+
29+
- run: sed -i 's/525568/16777471/' vendor/kwn/php-rdkafka-stubs/stubs/constants.php
2830

2931
- run: vendor/bin/phpunit --exclude-group=functional

pkg/rdkafka/Tests/fix_composer_json.php

-9
This file was deleted.

0 commit comments

Comments
 (0)