Fix seaneagan#73 by using package:supports_color

Author: seaneagan

CHANGELOG.md

@@ -1,4 +1,11 @@
-# 0.4.5
+
+## 0.4.6
+
+Features:
+
+  - Improve color-support detection (#73)
+
+## 0.4.5
 
 Features:
 

lib/src/plugins/help/help.dart

@@ -10,6 +10,7 @@ import 'package:unscripted/src/plugin.dart';
 import 'package:unscripted/src/usage.dart';
 import 'package:unscripted/src/util.dart';
 import 'package:quiver/strings.dart';
+import 'package:supports_color/supports_color.dart';
 
 import 'option_help.dart';
 import 'pens.dart';
@@ -79,7 +80,7 @@ class Help extends Plugin {
   }
 
   UsageFormatter _getUsageFormatter(Usage usage, bool isWindows) =>
-      new TerminalUsageFormatter(usage, shouldDisableColor(isWindows));
+      new TerminalUsageFormatter(usage, supportsColor);
 
   List<String> _getHelpPath(CommandInvocation commandInvocation) {
     var path = [];
@@ -102,6 +103,3 @@ class Help extends Plugin {
   }
 
 }
-
-// TODO: May need to also disable when testing help formatting output.
-bool shouldDisableColor(bool isWindows) => isWindows; // || !stdout.hasTerminal;

lib/src/plugins/help/usage_formatter.dart

@@ -21,7 +21,7 @@ class TerminalUsageFormatter extends UsageFormatter {
 
     var oldColorDisabled = color_disabled;
 
-    color_disabled = color;
+    color_disabled = !color;
 
     var parser = usage.parser;
     var description = usage.description;

pubspec.yaml

@@ -1,5 +1,5 @@
 name: unscripted
-version: 0.4.5
+version: 0.4.6
 author: Sean Eagan <[email protected]>
 description: Declarative command-line interface programming.
 homepage: http://github.com/seaneagan/unscripted
@@ -10,5 +10,6 @@ dependencies:
   mockable_filesystem: any
   path: any
   quiver: any
+  supports_color: any
 dev_dependencies:
   unittest: any