Closed pat-s closed 6 years ago
@larskotthoff ready for merge?
is the "a" here a typo?
Why do we need sudo here? Why not install the packages like in the main repo?
Why do we need sudo here? Why not install the packages like in the main repo?
There was a misunderstanding on my side with the whitelisted packages. Nevermind. Changed :)
is the "a" here a typo?
yes
ok lets merge when travis is finished and green
Should we update this now that mlr's travis is up and running?
Should we update this now that mlr's travis is up and running?
Yes, adapt it to mlr travis. I had not time in the last days, I'll try to update it asap
We're currently timing out. Can we have build stages here as well?
would be great if we could resolve this, yes. this is blocking us
I tried to split up the knitting by stages but it seems that resulting .md files are not carried over to the next stage? see this build building everything in one stage leads to timeout.
@mllg any ideas?
Would we be able to break it up into multiple chunks of .md files to be tested?
Would we be able to break it up into multiple chunks of .md files to be tested?
@SteveBronder thats what I tried (if I understand you correctly)
As we have working builds on trusty, I close this here.
see #114 and https://github.com/mlr-org/mlr/pull/1912#issuecomment-317062603