Expanding doctor command. (#417)

* Expanding doctor command. * Expanding doctor. * Fixing travis. * Forcing drush to error. * Travis bugs. * More travis witchery. * Fixing output. * Adding hirak.
acquia · Sep 19, 2016 · d2ac989 · d2ac989
1 parent bd85fe4
commit d2ac989
Show file tree

Hide file tree

Showing 11 changed files with 549 additions and 59 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -87,14 +87,19 @@ script:
   - blt acsf:init
   # Deploy build artifact.
   - blt deploy:build
+  # Ensure that the doctor doesn't report any problems at this point.
+  # We must temporarily move acquia/blt to require to appease the Doctor. It must then be removed so deploy cannot resolve the symlink later.
+  - composer remove acquia/blt --no-update --dev
+  - composer require acquia/blt --no-update
+  - composer global require "hirak/prestissimo:^0.3"
+  - blt doctor
+  - composer remove acquia/blt --no-update
   # Add Drupal VM config to repo.
   - blt vm:init
   # Switch back to 'blt' directory.
   - cd ../blt
   # Run 'blt' phpunit tests, excluding deploy-push tests.
   - phpunit tests/phpunit --group blt --exclude-group deploy-push
-  # Execute 'blt update' just to ensure that there isn't a non-zero exit code.
-  # - blt update
 
 deploy:
   provider: script

diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -2,13 +2,12 @@
 
 Please feel free to contribute to the project or file issues via the GitHub issue queue. When doing so, please keep the following points in mind:
 
+
 * BLT is distributed under the GPLv2 license; WITHOUT ANY WARRANTY.
 * The project maintainers make no commitment to respond to support requests,
   feature requests, or pull requests.
 * All contributions to BLT will be reviewed for compliance with Drupal Coding
   Standards and best practices as defined by the project maintainer.
-* Feature that are part of the [Road Map](https://github.com/acquia/blt/wiki/Road-Map)
-  will be prioritized for inclusion.
 
 BLT work is currently being tracked in the [BLT GitHub issue queue](https://github.com/acquia/blt/issues) and organized via a [Waffle.io Kanban Board](https://waffle.io/acquia/blt).
 
@@ -26,8 +25,6 @@ cd blt-project
 git init
 composer install
 ./vendor/bin/blt install-alias
-blt init
-blt configure
 rm -rf vendor
 composer update
 git add -A

diff --git a/composer.json b/composer.json
@@ -13,7 +13,8 @@
     "symfony/console":                  "~2",
     "symfony/twig-bridge": "~2",
     "php": ">=5.6",
-    "composer-plugin-api": "^1.0.0"
+    "composer-plugin-api": "^1.0.0",
+    "dflydev/dot-access-data": "^1.0"
   },
   "autoload": {
     "psr-4": {

diff --git a/composer.lock b/composer.lock
diff --git a/mkdocs.yml b/mkdocs.yml
@@ -36,5 +36,6 @@ pages:
           - Release notes: 'scripts/release-notes/README.md'
         - Setting up continuous integration: 'readme/ci.md'
         - Open source contribution: 'readme/os-contribution.md'
+  - Troubleshooting & Support: 'readme/support.md'
   - Extending / Overriding BLT: 'readme/extending-blt.md'
   - Contributing: 'CONTRIBUTING.md'
diff --git a/phing/tasks/deploy.xml b/phing/tasks/deploy.xml
@@ -13,6 +13,7 @@
         <echo message="Command should match syntax:"/>
         <echo message="blt deploy -Ddeploy.commitMsg='BLT-123: The commit message.' -Ddeploy.branch='master-build'"/>
         <fail message="Missing required parameters."/>
+        <!-- @todo create default values for these params. E.g., the last commit msg from the source branch, and "[source-branch]-build". -->
       </then>
     </if>
 

diff --git a/readme/support.md b/readme/support.md
@@ -0,0 +1,14 @@
+If you experience issues with a local BLT build, try using Dr. BLT to diagnose your problem:
+
+    blt doctor
+
+If that isn't helpful, please post an issue on the [GitHub issue queue](https://github.com/acquia/blt/issues) including the following information:
+
+- Your version of BLT, `composer info acquia/blt`
+- Your operating system
+- The full log output of your BLT command, wrapped in a [codeblock](https://help.github.com/articles/basic-writing-and-formatting-syntax/#quoting-code).
+
+In seeking help, please keep the following points in mind:
+
+* BLT is distributed under the GPLv2 license; WITHOUT ANY WARRANTY.
+* The project maintainers are under no obligation to respond to support requests, feature requests, or pull requests.
diff --git a/scripts/blt/alias b/scripts/blt/alias
@@ -10,5 +10,6 @@ function blt() {
     $GIT_ROOT/vendor/bin/blt "$@"
   else
     echo "You must run this command from within a BLT-generated project repository."
+    exit 1
   fi
 }
diff --git a/scripts/blt/ignore-existing.txt b/scripts/blt/ignore-existing.txt
@@ -1,7 +1,6 @@
 # These files from template dir will be included if they do not yet exist. They will not overwrite
 # existing files.
 .gitattributes
-.travis.yml
 project.yml
 composer.json
 README.md
-Original file line number
+Diff line change
@@ Expand Up / @@ -10,5 +10,6 @@ function blt() { @@
         $GIT_ROOT/vendor/bin/blt "$@"
       else
         echo "You must run this command from within a BLT-generated project repository."
+        exit 1
       fi
     }