Make pub lish validation gitignore-aware.

lib/src/validator/executable.dart

Index: lib/src/validator/executable.dart
diff --git a/lib/src/validator/executable.dart b/lib/src/validator/executable.dart
index 939b443470e8180dd1474c85853537d5359edb0a..ee52ce8799907870f26f797c65ea1c56cafc72f0 100644
--- a/lib/src/validator/executable.dart
+++ b/lib/src/validator/executable.dart
@@ -20,7 +20,8 @@ class ExecutableValidator extends Validator {
   Future validate() async {
     // TODO(rnystrom): This can print false positives since a script may be
     // produced by a transformer. Do something better.
-    var binFiles = entrypoint.root.listFiles(beneath: "bin", recursive: false)
+    var binFiles = entrypoint.root.listFiles(
+            beneath: "bin", recursive: false, useGitIgnore: true)

[Bob Nystrom, 2015/08/26 23:21:44]

Since this is copied in a few validators, how about a convenience method in
Validator for listing files?

[nweiz, 2015/08/27 19:07:56]

I think adding a wrapper function that just changes boolean defaults is more
confusing than it's worth. It makes it harder to reason about those defaults in
a global sense.

         .map((path) => entrypoint.root.relative(path))
         .toList();