Drop PHP7.3

[ChristophWurst]

Fixes #29258

Todo

• Adjust versioncheck.php
• Adjust composer.json and run composer update php
• Add to developer infos Fix min PHP7.4 in Nextcloud 24 documentation#9329

To be merged once 23 is branched off.

[come-nc]

It seems a lot of images in .drone.yml use PHP 7.3.

It also needs to be removed from .github/workflows/oci.yml

[nickvergessen]

It also needs to be removed from .github/workflows/oci.yml

I can use my script to drop this from all the app repositories.

[come-nc]

It would be good to finish and merge this before 8.1 is added to CI to avoid queuing too much jobs.

[skjnldsv]

@ChristophWurst ready when you are 😉

[ChristophWurst]

CI needs adjustment. Feel free to take over.

[come-nc]

The following images still need 7.4 equivalents:

image: ghcr.io/nextcloud/continuous-integration-acceptance-php7.3:acceptance-php7.3-3
      image: ghcr.io/nextcloud/continuous-integration-integration-php7.3:latest
  image: ghcr.io/nextcloud/continuous-integration-integration-php7.3:latest
  image: nextcloudci/litmus-php7.3:litmus-php7.3-1
  image: nextcloudci/php7.3-memcached:php7.3-memcached-3
  image: nextcloudci/samba-native-php7.3:samba-native-php7.3-1
  image: nextcloudci/samba-non-native-php7.3:samba-non-native-php7.3-1
  image: nextcloudci/webdav-apache-php7.3:webdav-apache-php7.3-3

[skjnldsv]

The following images still need 7.4 equivalents:

image: ghcr.io/nextcloud/continuous-integration-acceptance-php7.3:acceptance-php7.3-3
      image: ghcr.io/nextcloud/continuous-integration-integration-php7.3:latest
  image: ghcr.io/nextcloud/continuous-integration-integration-php7.3:latest
  image: nextcloudci/litmus-php7.3:litmus-php7.3-1
  image: nextcloudci/php7.3-memcached:php7.3-memcached-3
  image: nextcloudci/samba-native-php7.3:samba-native-php7.3-1
  image: nextcloudci/samba-non-native-php7.3:samba-non-native-php7.3-1
  image: nextcloudci/webdav-apache-php7.3:webdav-apache-php7.3-3

You can trigger them in https://github.com/nextcloud/docker-ci/actions/workflows/docker.yml

[come-nc]

You can trigger them in https://github.com/nextcloud/docker-ci/actions/workflows/docker.yml

I actually cannot: "Error: The user required level is not sufficient"

[nickvergessen]

I actually cannot: "Error: The user required level is not sufficient"

@come-nc you have been promoted :)

[come-nc]

@nickvergessen All tests are finally passing, except samba which is broken as well on master.

So what are the next steps to get this merged?

[nickvergessen]

I will bump all apps friday, after that is in we can merge this here

[nickvergessen]

#30896 Review time

[come-nc]

Good to merge then?
The only drone failure is the samba one and is not related.