nchammas / flintrock

A command-line tool for launching Apache Spark clusters.
Apache License 2.0
636 stars 116 forks source link

Simplify Travis CI script step #278

Closed nchammas closed 5 years ago

nchammas commented 5 years ago

The intention initially was to ensure that certain tests could run successfully without the maintainer requirements installed, which I suppose is a good goal, but in practice this setup has meant that I've missed adding new test files to the Travis CI build.

I think what I have in this PR is a better trade-off: Run the full test suite automatically so I don't have to manually update the Travis CI build, and take the small risk that I will introduce a maintainer dependency to the test suite without realizing it.