gildegoma/travis-build — comment on commit 980a539ee1510044064f4034dc136b188b4f7c96
File: lib/travis/build/job/test/scala.rb line 25
By gildegoma:

I fixed it in d90c1ebd7f2199451bf941c2a5f4a17fc6386c49 in the way you recommend, but I preferred to return "make test" like in Erlang and Node.js builders. I think that returning a maven command could be misleading and interpreted as a natural default for JVM languages (see also: b8be8504837f08c3fad8bbf5c6728d457dea44ef)

Currently there are 2 kinds of behaviors in builder implementations:

* Erlang and Node.js check build configuration in project and return "make test" when not found (as an "error command"),
* while other builders return their default command without any check in project files: Ruby (with rake, with bundle or not), PHP (with phpunit) and Clojure (with lein)

Actually both approaches are fine, but I wonder if we shouldn't unify them, or provide a more explicit way to raise that "build configuration not found", instead of "make test" execution ? I know, it's cosmetics that does not matter... but I preferred to drop the comment for memory ;-)