server icon indicating copy to clipboard operation
server copied to clipboard

Test PHP 8.2 through CI

Open come-nc opened this issue 3 years ago • 5 comments

The point of this PR is to test master with PHP 8.2 nightly/beta builds. Fix for the problems found should go in other PRs, and this one can be rebased on master to validate.

I may remove non-relevant CI from the branch to speed things up.

Signed-off-by: Côme Chilliet [email protected]

Contributes to https://github.com/nextcloud/server/issues/32595

come-nc avatar May 17 '22 16:05 come-nc

/rebase

come-nc avatar Jun 16 '22 12:06 come-nc

/rebase

come-nc avatar Jun 21 '22 11:06 come-nc

let's try to group the things that need fixing into issues, for the easy ones set the label "good first issue" for the tricky / urgent ones, we can discuss if we want to do them right away

PVince81 avatar Jun 21 '22 15:06 PVince81

/rebase

come-nc avatar Jun 28 '22 10:06 come-nc

/rebase

come-nc avatar Aug 02 '22 14:08 come-nc

/rebase

come-nc avatar Sep 05 '22 15:09 come-nc

/rebase

come-nc avatar Sep 08 '22 13:09 come-nc

There were 271 errors:

(Some are false-positive)

come-nc avatar Nov 14 '22 16:11 come-nc

https://github.com/nextcloud/server/pull/34819 should help

come-nc avatar Nov 14 '22 16:11 come-nc

/rebase

come-nc avatar Nov 15 '22 12:11 come-nc

There were 271 errors:

(Some are false-positive)

There were 119 errors:

20 are also there on 8.1 so are false positives.

come-nc avatar Nov 15 '22 13:11 come-nc

Opened https://github.com/sebastianbergmann/phpunit/issues/5099 on phpunit to figure out how to fix apps/oauth2/tests/Controller/OauthApiControllerTest.php

come-nc avatar Nov 15 '22 14:11 come-nc

/rebase

come-nc avatar Nov 15 '22 17:11 come-nc

  • mysql/8.1 Tests: 14461, Assertions: 37167, Errors: 14, Failures: 7, Warnings: 168, Skipped: 130. 1532
  • mysql/8.2 Tests: 14461, Assertions: 35960, Errors: 100, Failures: 89, Warnings: 149, Skipped: 130. 3159
  • sqlite/8.2 Tests: 14461, Assertions: 35960, Errors: 100, Failures: 89, Warnings: 149, Skipped: 130. 3161

come-nc avatar Nov 15 '22 17:11 come-nc

@come-nc as discussed yesterday, please add a list of checkboxes in the top post or the ticket so we get an overview what's left to fix and what challenges we have, for example updating Doctrine, etc.

if a topic gets bigger, like deciding what Doctrine version to pick for 8.2, please also raise a separate ticket to handle that topic separately and link to the checkbox.

thanks!

PVince81 avatar Nov 16 '22 08:11 PVince81

@come-nc as discussed yesterday, please add a list of checkboxes in the top post or the ticket so we get an overview what's left to fix and what challenges we have, for example updating Doctrine, etc.

if a topic gets bigger, like deciding what Doctrine version to pick for 8.2, please also raise a separate ticket to handle that topic separately and link to the checkbox.

thanks!

https://github.com/nextcloud/server/issues/32595#issuecomment-1315594229

come-nc avatar Nov 17 '22 11:11 come-nc

/rebase

come-nc avatar Nov 22 '22 14:11 come-nc

  • mysql/8.1 Tests: 14461, Assertions: 37167, Errors: 14, Failures: 7, Warnings: 168, Skipped: 130. 1532
  • mysql/8.2 Tests: 14461, Assertions: 35960, Errors: 100, Failures: 89, Warnings: 149, Skipped: 130. 3159
  • sqlite/8.2 Tests: 14461, Assertions: 35960, Errors: 100, Failures: 89, Warnings: 149, Skipped: 130. 3161
  • mysql/8.1 Tests: 14461, Assertions: 37168, Errors: 14, Failures: 7, Warnings: 168, Skipped: 130. 1532
  • mysql/8.2 Tests: 14461, Assertions: 35966, Errors: 99, Failures: 89, Warnings: 149, Skipped: 130. 3147
  • sqlite/8.2 Tests: 14461, Assertions: 35965, Errors: 99, Failures: 89, Warnings: 149, Skipped: 130. 3149

Only one error fixed :-(

come-nc avatar Nov 22 '22 15:11 come-nc

/rebase

come-nc avatar Nov 28 '22 13:11 come-nc

  • mysql/8.1 Tests: 14461, Assertions: 37167, Errors: 14, Failures: 7, Warnings: 168, Skipped: 130. 1532
  • mysql/8.2 Tests: 14461, Assertions: 35960, Errors: 100, Failures: 89, Warnings: 149, Skipped: 130. 3159
  • sqlite/8.2 Tests: 14461, Assertions: 35960, Errors: 100, Failures: 89, Warnings: 149, Skipped: 130. 3161
  • mysql/8.1 Tests: 14461, Assertions: 37168, Errors: 14, Failures: 7, Warnings: 168, Skipped: 130. 1532
  • mysql/8.2 Tests: 14461, Assertions: 35966, Errors: 99, Failures: 89, Warnings: 149, Skipped: 130. 3147
  • sqlite/8.2 Tests: 14461, Assertions: 35965, Errors: 99, Failures: 89, Warnings: 149, Skipped: 130. 3149

Only one error fixed :-(

  • mysql/8.1 Tests: 14465, Assertions: 37184, Errors: 13, Failures: 7, Warnings: 168, Skipped: 130. 1526
  • mysql/8.2 Tests: Tests: 14465, Assertions: 36344, Errors: 75, Failures: 99, Warnings: 150, Skipped: 130. 2896

come-nc avatar Nov 28 '22 14:11 come-nc

rebased

come-nc avatar Dec 19 '22 13:12 come-nc

(rebased)

come-nc avatar Jan 23 '23 15:01 come-nc