Merge pull request #166 from infosiftr/php7.3

Update Drupal 7 to PHP 7.3

Author: yosifkit

7/apache/Dockerfile

@@ -1,5 +1,5 @@
 # from https://www.drupal.org/docs/8/system-requirements/drupal-8-php-requirements
-FROM php:7.2-apache-stretch
+FROM php:7.3-apache-stretch
 # TODO switch to buster once https://github.com/docker-library/php/issues/865 is resolved in a clean way (either in the PHP image or in PHP itself)
 
 # install the PHP extensions we need

7/fpm-alpine/Dockerfile

@@ -1,5 +1,5 @@
 # from https://www.drupal.org/docs/8/system-requirements/drupal-8-php-requirements
-FROM php:7.2-fpm-alpine
+FROM php:7.3-fpm-alpine
 
 # install the PHP extensions we need
 # postgresql-dev is needed for https://bugs.alpinelinux.org/issues/3642

7/fpm/Dockerfile

@@ -1,5 +1,5 @@
 # from https://www.drupal.org/docs/8/system-requirements/drupal-8-php-requirements
-FROM php:7.2-fpm-stretch
+FROM php:7.3-fpm-stretch
 # TODO switch to buster once https://github.com/docker-library/php/issues/865 is resolved in a clean way (either in the PHP image or in PHP itself)
 
 # install the PHP extensions we need

update.sh

@@ -9,9 +9,11 @@ if [ ${#versions[@]} -eq 0 ]; then
 fi
 versions=( "${versions[@]%/}" )
 
+# https://www.drupal.org/docs/8/system-requirements/php-requirements#php_required
 defaultPhpVersion='7.3'
 declare -A phpVersions=(
-	[7]='7.2'
+	# https://www.drupal.org/docs/7/system-requirements/php-requirements#php_required
+	#[7]='7.2'
 )
 
 travisEnv=