Skip to content

Conversation

@PVince81
Copy link
Member

Should fix this newly appeared issue: https://github.com/nextcloud/guests/pull/510/checks?check_run_id=1557739738

Code coverage needs to be enabled in php.ini by setting 'xdebug.mode' to 'coverage'

The change is taken from the mail app.

@PVince81 PVince81 added the 3. to review Waiting for reviews label Dec 15, 2020
@PVince81 PVince81 self-assigned this Dec 15, 2020
@PVince81
Copy link
Member Author

ok, one step further, now it's the oci one that is unhappy about composer deps issues ?! https://github.com/nextcloud/guests/pull/511/checks?check_run_id=1558104190

@PVince81 PVince81 force-pushed the bugfix/noid/add-coverage-fix-ci branch from 7a00cf1 to 6a6c4a0 Compare December 15, 2020 15:53
@PVince81
Copy link
Member Author

I've disabled coverage now as I saw it was explicitly disabled elsewhere and there is anyway no codecov setup so far.
Let's hope this magically fix the deps for oci...

Aligned phpunit oci setup with the one from mysql.
Removed phpunit version.

Signed-off-by: Vincent Petry <[email protected]>
@PVince81 PVince81 force-pushed the bugfix/noid/add-coverage-fix-ci branch from bdf83ef to 346ddc1 Compare December 16, 2020 08:49
@juliusknorr juliusknorr merged commit 22f5351 into master Dec 16, 2020
@juliusknorr juliusknorr deleted the bugfix/noid/add-coverage-fix-ci branch December 16, 2020 12:58
@rullzer
Copy link
Member

rullzer commented Feb 2, 2021

/backport to stable-1.5

@backportbot-nextcloud backportbot-nextcloud bot added the backport-request Pending backport by the backport-bot label Feb 2, 2021
@backportbot-nextcloud
Copy link

The backport to stable-1.5 failed. Please do this backport manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews backport-request Pending backport by the backport-bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants