Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

Lines changed: 4 additions & 2 deletions
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

Lines changed: 0 additions & 9 deletions
This file was deleted.

0 commit comments

Comments
(0)

AltStyle によって変換されたページ (->オリジナル) /