morganey-lang / Morganey

Lambda Calculus based, interpreted programming language that recognizes Church encoded structures.
43 stars 4 forks source link

Make stdlib subproject build morganey module index #322

Closed keddelzz closed 7 years ago

keddelzz commented 7 years ago

Generate the module-index of Morganey during the sbt-task 'build'.

closes #304

keddelzz commented 7 years ago

It seems like the build timed out. I'm not sure why this happens, though.

rexim commented 7 years ago

@keddelzz yeah I restarted it. Will see if it passes this time. :)

keddelzz commented 7 years ago

@rexim, Thank's a lot!

rexim commented 7 years ago

@ForNeVeR thanks for the review! @keddelzz thanks for the changes!

ForNeVeR commented 7 years ago

The tests are timed out on both of the CI servers. That's no coincidence; something really bad's happening. These're Times of Ending everyone is talking about...

rexim commented 7 years ago

I'm gonna check if the CI feels better tomorrow.

rexim commented 7 years ago

@keddelzz careful, I'm pushing to your branch.

rexim commented 7 years ago

@keddelzz do I understand correcly that the build command simply copies all of the jars to a single place for the sake of convenience and doesn't affect the compilation and testing processes anyhow?

keddelzz commented 7 years ago

@keddelzz do I understand correcly that the build command simply copies all of the jars to a single place for the sake of convenience and doesn't affect the compilation and testing processes?

@rexim Exactly!

rexim commented 7 years ago

@keddelzz great! Thank you very much! Please review my additional changes when you get a chance.

rexim commented 7 years ago

@keddelzz ah, since you're the PR author you probably cannot use the GitHub's review mechanism. But please feel free to leave any comments if you feel something's off. :)

keddelzz commented 7 years ago

@rexim I approve your changes. Thank you very much for looking into the failed CI process!

rexim commented 7 years ago

@keddelzz @ForNeVeR thanks guys for the review!