Skip to content

Commit

Permalink
Turn on assertions
Browse files Browse the repository at this point in the history
Turning them of results in a drop in coverage. Also, it is better to
execute that code in case it is wrong.
  • Loading branch information
greg0ire committed Oct 15, 2020
1 parent f78bc33 commit 8ee26df
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion .github/workflows/continuous-integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ jobs:
with:
php-version: "${{ matrix.php-version }}"
coverage: "pcov"
ini-values: "zend.assertions=1"

- name: "Cache dependencies installed with composer"
uses: "actions/cache@v2"
Expand Down Expand Up @@ -93,6 +94,7 @@ jobs:
php-version: "${{ matrix.php-version }}"
extensions: "oci8"
coverage: "pcov"
ini-values: "zend.assertions=1"

- name: "Cache dependencies installed with composer"
uses: "actions/cache@v2"
Expand Down Expand Up @@ -141,6 +143,7 @@ jobs:
php-version: "${{ matrix.php-version }}"
extensions: "pdo_oci"
coverage: "pcov"
ini-values: "zend.assertions=1"

- name: "Cache dependencies installed with composer"
uses: "actions/cache@v2"
Expand Down Expand Up @@ -204,6 +207,7 @@ jobs:
with:
php-version: "${{ matrix.php-version }}"
coverage: "pcov"
ini-values: "zend.assertions=1"

- name: "Cache dependencies installed with composer"
uses: "actions/cache@v2"
Expand Down Expand Up @@ -268,6 +272,7 @@ jobs:
with:
php-version: "${{ matrix.php-version }}"
coverage: "pcov"
ini-values: "zend.assertions=1"
extensions: "${{ matrix.extension }}"

- name: "Cache dependencies installed with composer"
Expand Down Expand Up @@ -343,6 +348,7 @@ jobs:
with:
php-version: "${{ matrix.php-version }}"
coverage: "pcov"
ini-values: "zend.assertions=1"
extensions: "${{ matrix.extension }}"

- name: "Cache dependencies installed with composer"
Expand Down Expand Up @@ -416,6 +422,7 @@ jobs:
with:
php-version: "${{ matrix.php-version }}"
coverage: "pcov"
ini-values: "zend.assertions=1"
tools: "pecl"
extensions: "${{ matrix.extension }}-5.7.0preview"

Expand Down Expand Up @@ -478,7 +485,7 @@ jobs:
with:
php-version: "${{ matrix.php-version }}"
coverage: "pcov"
ini-values: "extension=ibm_db2.so, ibm_db2.instance_name=db2inst1"
ini-values: "zend.assertions=1,extension=ibm_db2.so, ibm_db2.instance_name=db2inst1"

- name: "Install ibm_db2 extension"
run: "ci/github/ext/install-ibm_db2.sh ${{ matrix.php-version }}"
Expand Down

0 comments on commit 8ee26df

Please sign in to comment.