Browse Source

fix(ci): Disable coverage on pull requests

Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de>
Co-authored-by: Joas Schilling <213943+nickvergessen@users.noreply.github.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Ferdinand Thiessen 2 months ago
parent
commit
f546a2fe14

+ 2 - 2
.github/workflows/files-external-s3.yml

@@ -42,8 +42,8 @@ jobs:
       matrix:
         php-versions: ['8.0', '8.1', '8.2', '8.3']
         include:
-          - php-versions: '8.3'
-            coverage: true
+          - php-versions: '8.2'
+            coverage: ${{ github.event_name != 'pull_request' }}
 
     name: php${{ matrix.php-versions }}-s3
 

+ 2 - 2
.github/workflows/files-external-webdav.yml

@@ -42,8 +42,8 @@ jobs:
       matrix:
         php-versions: ['8.0', '8.1', '8.2', '8.3']
         include:
-          - php-versions: '8.3'
-            coverage: true
+          - php-versions: '8.2'
+            coverage: ${{ github.event_name != 'pull_request' }}
 
     name: php${{ matrix.php-versions }}-webdav
 

+ 3 - 3
autotest-external.sh

@@ -35,7 +35,7 @@ if ! [ -x "$PHPUNIT" ]; then
 	if [ -x "$PWD/build/integration/vendor/phpunit/phpunit/phpunit" ]; then
 		PHPUNIT="$PWD/build/integration/vendor/phpunit/phpunit/phpunit"
 	else
-		echo "phpunit executable not found, please install phpunit version >= 9.6" >&2
+		echo "phpunit executable not found, please install phpunit version >= 9.0" >&2
 		exit 3
 	fi
 fi
@@ -44,8 +44,8 @@ PHPUNIT_VERSION=$("$PHPUNIT" --version | cut -d" " -f2)
 PHPUNIT_MAJOR_VERSION=$(echo $PHPUNIT_VERSION | cut -d"." -f1)
 PHPUNIT_MINOR_VERSION=$(echo $PHPUNIT_VERSION | cut -d"." -f2)
 
-if ! [ $PHPUNIT_MAJOR_VERSION -gt 9 -o \( $PHPUNIT_MAJOR_VERSION -eq 9 -a $PHPUNIT_MINOR_VERSION -ge 6 \) ]; then
-	echo "phpunit version >= 9.6 required. Version found: $PHPUNIT_VERSION" >&2
+if ! [ $PHPUNIT_MAJOR_VERSION -gt 9 -o \( $PHPUNIT_MAJOR_VERSION -eq 9 -a $PHPUNIT_MINOR_VERSION -ge 0 \) ]; then
+	echo "phpunit version >= 9.0 required. Version found: $PHPUNIT_VERSION" >&2
 	exit 4
 fi