Split packages on Travis (#307)

* Run each package in it's on Travis job
* Check dartfmt of each package
* Only build on the master branch
* Cache packages
* dartfmt all files

Author: kevmoo

.travis.yml

@@ -4,4 +4,18 @@ dart:
   - dev
   - stable
 
+env:
+  - PKG=build_test
+  - PKG=build
+  - PKG=build_runner
+  - PKG=build_barback
+
 script: ./tool/travis.sh
+
+# Only building master means that we don't run two builds for each pull request.
+branches:
+  only: [master]
+
+cache:
+ directories:
+   - $HOME/.pub-cache

build/lib/src/builder/build_step.dart

@@ -49,7 +49,8 @@ abstract class BuildStep implements AssetReader, AssetWriter {
   /// Future since the runner will be responsible for waiting until all outputs
   /// are written.
   @override
-  Future writeAsString(AssetId id, FutureOr<String> contents, {Encoding encoding: UTF8});
+  Future writeAsString(AssetId id, FutureOr<String> contents,
+      {Encoding encoding: UTF8});
 
   @Deprecated('Use `canRead` instead')
   Future<bool> hasInput(AssetId id);

build_barback/test/analyzer/resolver_test.dart

@@ -303,7 +303,9 @@ class TestBuilder extends Builder {
   TestBuilder(this.validator);
 
   @override
-  final buildExtensions = const {'': const ['.unused']};
+  final buildExtensions = const {
+    '': const ['.unused']
+  };
 
   @override
   Future build(BuildStep buildStep) async {

build_runner/lib/src/generate/watch_impl.dart

@@ -126,8 +126,7 @@ class WatchImpl {
       }
 
       _logger.info('Starting next build');
-      _currentBuild =
-          _buildImpl.runBuild(updates: updatedInputs);
+      _currentBuild = _buildImpl.runBuild(updates: updatedInputs);
       _currentBuild.then((result) {
         // Terminate the watcher if the build script is updated, there is no
         // need to continue listening.

build_runner/test/asset/file_based_test.dart

@@ -36,14 +36,13 @@ void main() {
 
     test('can check for existence of any application package files', () async {
       expect(await reader.canRead(makeAssetId('basic_pkg|hello.txt')), isTrue);
-      expect(await reader.canRead(makeAssetId('basic_pkg|lib/hello.txt')),
-          isTrue);
-      expect(await reader.canRead(makeAssetId('basic_pkg|web/hello.txt')),
-          isTrue);
+      expect(
+          await reader.canRead(makeAssetId('basic_pkg|lib/hello.txt')), isTrue);
+      expect(
+          await reader.canRead(makeAssetId('basic_pkg|web/hello.txt')), isTrue);
 
       expect(await reader.canRead(makeAssetId('basic_pkg|a.txt')), isFalse);
-      expect(
-          await reader.canRead(makeAssetId('basic_pkg|lib/a.txt')), isFalse);
+      expect(await reader.canRead(makeAssetId('basic_pkg|lib/a.txt')), isFalse);
     });
 
     test('can check for existence of package dependency files in lib',

tool/travis.sh

@@ -7,25 +7,8 @@
 # Fast fail the script on failures.
 set -e
 
-pushd build_test
+pushd $PKG
 pub upgrade
-dartanalyzer --fatal-warnings lib/build_test.dart
-popd
-
-pushd build
-pub upgrade
-dartanalyzer --fatal-warnings lib/build.dart
-pub run test
-popd
-
-pushd build_runner
-pub upgrade
-dartanalyzer --fatal-warnings lib/build_runner.dart
-pub run test
-popd
-
-pushd build_barback
-pub upgrade
-dartanalyzer --fatal-warnings lib/build_barback.dart
-pub run test
-popd
+dartanalyzer --fatal-warnings .
+echo Any unformatted files?
+dartfmt -n --set-exit-if-changed .