Adding Travis CI and Coveralls support

Adding Travis CI and Coveralls support

only use compact config if we're not doing code coverage

R=sigmund@google.com

Committed: https://github.com/dart-lang/dev_compiler/commit/208251874f9caffc8e4be912572f0727e7329e5d

[kevmoo, 2015-03-17 01:46:55]

kevmoo@google.com changed reviewers:
+ jmesserly@google.com, sigmund@google.com

[Siggi Cherem (dart-lang), 2015-03-17 15:03:50]

This is really nice!

Some small comments below, but otherwise LGTM.

https://codereview.chromium.org/1014573003/diff/1/pubspec.yaml
File pubspec.yaml (right):

https://codereview.chromium.org/1014573003/diff/1/pubspec.yaml#newcode6
pubspec.yaml:6: url: https://github.com/dart-lang/ddc_analyzer.git
If this were to still require a password, another option we could do is to
automate this edit you have here, and do it in travis. For example, add a local
.gitconfig file to rewrite these or make it part of the travis script to change
it and call pub-get on it. Thoughts?

https://codereview.chromium.org/1014573003/diff/1/tool/travis.sh
File tool/travis.sh (right):

https://codereview.chromium.org/1014573003/diff/1/tool/travis.sh#newcode9
tool/travis.sh:9: git clean -fdx
I had not idea this existed, that's pretty cool!

https://codereview.chromium.org/1014573003/diff/1/tool/travis.sh#newcode16
tool/travis.sh:16: dart --checked test/all_tests.dart
could we call `source test/test.sh` here instead? it runs "all_tests.dart" but
also validates the output of the compiler, runs the analyzer, and checks that
all files are formatted correctly.

[kevmoo, 2015-03-17 15:14:39]

https://codereview.chromium.org/1014573003/diff/1/pubspec.yaml
File pubspec.yaml (right):

https://codereview.chromium.org/1014573003/diff/1/pubspec.yaml#newcode6
pubspec.yaml:6: url: https://github.com/dart-lang/ddc_analyzer.git
On 2015/03/17 15:03:50, Siggi Cherem (dart-lang) wrote:
> If this were to still require a password, another option we could do is to
> automate this edit you have here, and do it in travis. For example, add a
local
> .gitconfig file to rewrite these or make it part of the travis script to
change
> it and call pub-get on it. Thoughts?

Now that this dependency is public, we shouldn't have a problem. This URL is
only used by pub, so the fact that it doesn't use SSH is not an issue – right?

https://codereview.chromium.org/1014573003/diff/1/tool/travis.sh
File tool/travis.sh (right):

https://codereview.chromium.org/1014573003/diff/1/tool/travis.sh#newcode9
tool/travis.sh:9: git clean -fdx
On 2015/03/17 15:03:50, Siggi Cherem (dart-lang) wrote:
> I had not idea this existed, that's pretty cool!

:-)

https://codereview.chromium.org/1014573003/diff/1/tool/travis.sh#newcode16
tool/travis.sh:16: dart --checked test/all_tests.dart
On 2015/03/17 15:03:50, Siggi Cherem (dart-lang) wrote:
> could we call `source test/test.sh` here instead? it runs "all_tests.dart" but
> also validates the output of the compiler, runs the analyzer, and checks that
> all files are formatted correctly.

Yes, but let me get this in first.

I was having trouble getting tesh.sh to pass on travis. Wanted to get something
in first.

[Siggi Cherem (dart-lang), 2015-03-17 15:18:07]

lgtm

https://codereview.chromium.org/1014573003/diff/1/pubspec.yaml
File pubspec.yaml (right):

https://codereview.chromium.org/1014573003/diff/1/pubspec.yaml#newcode6
pubspec.yaml:6: url: https://github.com/dart-lang/ddc_analyzer.git
On 2015/03/17 15:14:38, kevmoo wrote:
> On 2015/03/17 15:03:50, Siggi Cherem (dart-lang) wrote:
> > If this were to still require a password, another option we could do is to
> > automate this edit you have here, and do it in travis. For example, add a
> local
> > .gitconfig file to rewrite these or make it part of the travis script to
> change
> > it and call pub-get on it. Thoughts?
> 
> Now that this dependency is public, we shouldn't have a problem. This URL is
> only used by pub, so the fact that it doesn't use SSH is not an issue
– right?

Let's try it, I think you are right.

[kevmoo, 2015-03-17 15:20:00]

Committed patchset #3 (id:40001) manually as
208251874f9caffc8e4be912572f0727e7329e5d (presubmit successful).

[Jennifer Messerly, 2015-03-17 15:30:12]

https://codereview.chromium.org/1014573003/diff/1/tool/travis.sh
File tool/travis.sh (right):

https://codereview.chromium.org/1014573003/diff/1/tool/travis.sh#newcode16
tool/travis.sh:16: dart --checked test/all_tests.dart
On 2015/03/17 15:14:38, kevmoo wrote:
> On 2015/03/17 15:03:50, Siggi Cherem (dart-lang) wrote:
> > could we call `source test/test.sh` here instead? it runs "all_tests.dart"
but
> > also validates the output of the compiler, runs the analyzer, and checks
that
> > all files are formatted correctly.
> 
> Yes, but let me get this in first.
> 
> I was having trouble getting tesh.sh to pass on travis. Wanted to get
something
> in first.

if it helps, we can move the diff'ing of expected output into the Dart test
code.
but test.sh is still nice because it runs the analyzer :)