Skip to content
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

Merged
merged 2 commits into from
Nov 5, 2020
Merged

Bump play 284 #116

merged 2 commits into from
Nov 5, 2020

Conversation

ignasi35
Copy link
Member

@ignasi35 ignasi35 commented Nov 5, 2020

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
Copy link
Member Author

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.

Copy link
Contributor

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?

Copy link
Member Author

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.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

raised #118

@ignasi35 ignasi35 requested a review from ennru November 5, 2020 12:00
@ignasi35 ignasi35 merged commit ccfa34a into playframework:2.8.x Nov 5, 2020
@ignasi35 ignasi35 deleted the bump-play-284 branch November 5, 2020 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants