Travis builds failing due to build timeouts about purescript HOT 10 CLOSED

We should be able to add output/ to the Travis cache.

from purescript.

Thanks Phil - with the additions in icyrockcom@b2d0f09 the last Travis build now takes 7 mins (though without the latest word-count exercise that's pending approval) vs 21 / 23 / 24 / 34 mins.

This is a nice speed up, but not as much as I hoped so. Does it make sense to cache bower_components, too?

from purescript.

Yes, we can cache bower_components, but I'm not sure how much it will speed things up.

from purescript.

Latest build of the above icyrockcom@3226c6e finished in 3:39. While the variations are obviously great, this version promises to be perhaps 25-50% faster, so I'd go with it. If nothing else, there's less noise in the build logs.

from purescript.

Any comments on this one or changes I should make? If this looks good, I can rebase on top of the current master for you to merge, let me know.

from purescript.

Looks good to me, thanks :)

from purescript.

Thanks @lpil, rebased. The build finished in 7:09, so quite slower than the other day, but still quite faster than current builds. For some reason, the project is being built even though it should be in the cache, e.g.:

-------------------------------------------------------
Testing accumulate
cache: lrwxrwxrwx 1 travis travis 56 Mar 14 22:59 bower_components -> /home/travis/.exercise_cache/accumulate/bower_components
cache: lrwxrwxrwx 1 travis travis 46 Mar 14 22:59 output -> /home/travis/.exercise_cache/accumulate/output
* Building project in /home/travis/build/icyrockcom/xpurescript/exercises/accumulate
Compiling Accumulate
Compiling Test.Main

so we are not getting the full speedup here. I could not reproduce locally, I'll have to look into it.

If you have a moment, please merge into master, as current builds are failing. For example, @paf31 merged word-count the other day, but the build failed after 50 minutes.

from purescript.

Can you please make a PR?

from purescript.

@paf31 Sure, #24

from purescript.

Closed by #24

from purescript.