-
-
Notifications
You must be signed in to change notification settings - Fork 549
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
Bump play 284 #116
Bump play 284 #116
Conversation
pushd play-scala-streaming-example && scripts/test-sbt && popd | ||
# uses vanilla sbt | ||
# pushd play-scala-tls-example && scripts/test-sbt && popd | ||
pushd play-scala-websocket-example && scripts/test-sbt && popd |
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'm sneaking this one in as an early attempt to quickly run samples locally for faster diagnosis in case of regression.
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.
We could add something like https://github.com/akka/akka-platform-guide/blob/main/scripts/format-all.sh and even remove the separate Travis jobs.
WDYT?
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.
We could add something like https://github.com/akka/akka-platform-guide/blob/main/scripts/format-all.sh and even remove the separate Travis jobs.
WDYT?
Hmm, yeah, simplifying this code is a good idea.
I'm not sure about running all tests in a single travis job. Every once in a while there's a failure here or there and having separate jobs makes it faster to diagnose. Let's merge this as is and discuss improvements in a new issue.
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.
raised #118
playframework/play-meta#166