-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Make dotty use scala-infra for CI. #591
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@@ -0,0 +1,6 @@ | |||
baseDir=${WORKSPACE-`pwd`} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think these scripts will need #!/bin/bash -e
to ensure the job fails.
/rebuild |
. $scriptsDir/common | ||
|
||
update scala scala | ||
sbt -Ddotty.travis.build=yes update compile test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-no-colors
will avoid ugly ANSI codes in the output.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, I've just seen scala/scala-jenkins-infra#57.
Allows sbts not to lock on the same ivy cache.
@odersky please review and merge it to unblock other PRs |
@DarkDimius Not sure if you say my comment above, but I don't think you are correctly propagating the exit codes of the
|
@retronym I can add |
See discussion in scala#591
Make dotty use scala-infra for CI.
The |
@adriaanm thanks for explaining! |
@adriaanm Thanks for huge help in getting this working 👍