refactor: add version constraint v1.24.3, reformat files (#73)

Author: stasadev

config.browsersync.yaml

@@ -1,6 +1,6 @@
 #ddev-generated
 web_extra_exposed_ports:
-  - name: browsersync
-    container_port: 3000
-    http_port: 3001
-    https_port: 3000
+    - name: browsersync
+      container_port: 3000
+      http_port: 3001
+      https_port: 3000

images/template-button.png

@@ -1,13 +1,5 @@
 name: ddev-browsersync
 
-pre_install_actions:
-- |
-  #ddev-nodisplay
-  #ddev-description:Check for minimum DDEV requirement (v1.19.3)
-  if ! ( ddev debug capabilities 2>/dev/null | grep multiple-dockerfiles >/dev/null 2>&1 ) ; then
-    echo "This add-on requires DDEV v1.19.3 or higher, please upgrade." && exit 2
-  fi
-
 project_files:
   - config.browsersync.yaml
   - web-build/Dockerfile.ddev-browsersync
@@ -17,26 +9,35 @@ project_files:
   - scripts/remove-wordpress-settings.sh
   - scripts/setup-wordpress-settings.sh
 
+ddev_version_constraint: '>= v1.24.3'
+
 post_install_actions:
-- |
-  #ddev-nodisplay
-  #ddev-description:Check for global 'browser-sync.js'
-  if ( test -f "$HOME/.ddev/commands/web/browsersync" ) ; then
-    echo "Note: '~/.ddev/commands/web/browsersync' is no longer required and can be safely removed." && exit 0
-  fi
-  #ddev-description:Check for 'browser-sync.js'
-  if ( test -f "$DDEV_APPROOT/commands/web/browsersync.js" ) ; then
-    echo "Note: '$DDEV_APPROOT/commands/web/browsersync.js' is no longer required and can be safely removed." && exit 0
-  fi
-  #ddev-nodisplay
-  #ddev-description:Remove old 'docker-compose.browsersync.yaml'
-  if grep "#ddev-generated" $DDEV_APPROOT/.ddev/docker-compose.browsersync.yaml 2>/dev/null; then rm -f "$DDEV_APPROOT/.ddev/docker-compose.browsersync.yaml"; fi
-  #ddev-description:Install browsersync settings for WordPress if applicable
-  scripts/setup-wordpress-settings.sh
+  - |
+    #ddev-nodisplay
+    #ddev-description:Check for global browsersync
+    if test -f "$HOME/.ddev/commands/web/browsersync"; then
+      echo "Note: '~/.ddev/commands/web/browsersync' is no longer required and can be safely removed." && exit 0
+    fi
+  - |
+    #ddev-nodisplay
+    #ddev-description:Check for 'browser-sync.js'
+    if test -f "$DDEV_APPROOT/commands/web/browsersync.js"; then
+      echo "Note: '$DDEV_APPROOT/commands/web/browsersync.js' is no longer required and can be safely removed." && exit 0
+    fi
+  - |
+    #ddev-nodisplay
+    #ddev-description:Remove old 'docker-compose.browsersync.yaml'
+    if grep "#ddev-generated" $DDEV_APPROOT/.ddev/docker-compose.browsersync.yaml 2>/dev/null; then
+      rm -f "$DDEV_APPROOT/.ddev/docker-compose.browsersync.yaml"
+    fi
+  - |
+    #ddev-nodisplay
+    #ddev-description:Install browsersync settings for WordPress if applicable
+    scripts/setup-wordpress-settings.sh
 
 removal_actions:
-- |
-  #ddev-nodisplay
-  #ddev-description:Remove browsersync settings for WordPress if applicable
-  rm -f "${DDEV_APPROOT}/wp-config-ddev-browsersync.php"
-  scripts/remove-wordpress-settings.sh
+  - |
+    #ddev-nodisplay
+    #ddev-description:Remove browsersync settings for WordPress if applicable
+    rm -f "${DDEV_APPROOT}/wp-config-ddev-browsersync.php"
+    scripts/remove-wordpress-settings.sh