ISSUE-204: final changes

.github/workflows/TestAcquia.yml

@@ -53,7 +53,8 @@ jobs:
           # Recommended settings for Acquia
           ddev composer config --no-plugins allow-plugins.acquia/drupal-recommended-settings false
           ddev composer require acquia/drupal-recommended-settings
-          # Apply Acquia recommended settings manually, before DDEV settings.
+
+          # Acquia recommended settings before DDEV settings.
           sed -i '/\/\/ Automatically generated include for settings managed by ddev./i require DRUPAL_ROOT . \
             "/../vendor/acquia/drupal-recommended-settings/settings/acquia-recommended.settings.php";' \
             docroot/sites/default/settings.php
@@ -67,8 +68,9 @@ jobs:
           echo "\$settings['config_sync_directory'] = '../config';" >> web/sites/default/settings.php
           ddev drush config:export -y
 
-          # When config is moved to ../config, the .htaccess to avoid execution
-          # is not create automatically. Running cron once does that for us.
+          # When config is moved to ../config, the .htaccess to avoid code
+          # execution is not created automatically. Running cron once creates
+          # the needed .htaccess file which improves security.
           ddev drush cron
 
       - name: Create .drainpipeignore

scaffold/github/actions/acquia/deploy/action.yml

@@ -73,8 +73,8 @@ runs:
 
     - name: Take site out of Maintenance Mode
       run: |
-        source .github/actions/drainpipe/set-env/bash_aliases
 
+        source .github/actions/drainpipe/set-env/bash_aliases
         drainpipe_exec "./vendor/bin/task drupal:maintenance:off site=@${{ inputs.environment }}"
       shell: bash
 

tasks/drupal.yml

@@ -85,4 +85,4 @@ tasks:
   maintenance:off:
     desc: Turn off Maintenance Mode
     cmds:
-      - ./vendor/bin/drush {{.site}}  --yes state:set system.maintenance_mode 0 --input-format=integer
+      - ./vendor/bin/drush {{.site}} --yes state:set system.maintenance_mode 0 --input-format=integer