search

cstjean / TraceCalls.jl

A debugging and profiling tool for Julia
Other
46 stars 4 forks source link

Try @iblis17's suggestion for the travis time out #48

Closed cstjean closed 7 years ago

cstjean commented 7 years ago

I hopefully got the syntax right. See also https://github.com/travis-ci/travis-ci/issues/7431

codecov-io commented 7 years ago

Codecov Report

Merging #48 into master will decrease coverage by 55.26%. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #48       +/-   ##
========================================
- Coverage   55.26%    0%   -55.27%     
========================================
  Files           2     2               
  Lines         532   516       -16     
========================================
- Hits          294     0      -294     
- Misses        238   516      +278
Impacted Files Coverage Δ
src/code_update.jl 0% <0%> (-88.99%) :arrow_down:
src/TraceCalls.jl 0% <0%> (-45.66%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2967be2...08b2feb. Read the comment docs.

cstjean commented 7 years ago

It seems to have exited immediately with success without running any of the tests.

iblislin commented 7 years ago

hmm ... no idea why

the sample of default testing commands: https://travis-ci.org/cstjean/TraceCalls.jl/jobs/272625174#L483

maybe this?

- julia --color=yes -e 'Pkg.clone(pwd())'
- julia --color=yes -e "Pkg.build(\"${JL_PKG}\")"
- travis_wait 40 julia --check-bounds=yes --color=yes -e "Pkg.test(\"${JL_PKG}\", coverage=true)"
cstjean commented 7 years ago

Doesn't seem to work either...

I fixed the issue on master by adding a print statement to my user-guide notebook. It's an ugly compromise, but it works. I'll ask tkelman about travis-wait next time I tag. Thank you for your help!

I'm sure there's a way in Julia to launch a background thread that prints to STDERR every 5 minutes for 40 minutes. That might be much easier than fighting travis.