-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
TemplateImports cleanup #8403
TemplateImports cleanup #8403
Conversation
@@ -25,7 +25,7 @@ | |||
)); | |||
|
|||
static { | |||
List<String> minimalJavaImports = new ArrayList(); | |||
List<String> minimalJavaImports = new ArrayList<String>(); |
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.
You can use diamond operator here because it is Java 7+ code
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 don't think the build-link project is. Here's what I get if I try:
[error] playframework/framework/src/build-link/src/main/java/play/TemplateImports.java:28: diamond operator is not supported in -source 1.6
[error] (use -source 7 or higher to enable diamond operator)
[error] List<String> minimalJavaImports = new ArrayList<>();
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.
Got it
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.
👍
Thank you, @benmccann. Just our curiosity, in what part of Gradle integration are you currently working? Best. |
Just rebasing some old PRs I opened a long time ago |
Backport here: #8414. |
* Documentation test task depending on publishLocal for play-docs (#8353) We cannot have `apiURL` scoped since later sbt reads the setting without a scope. So the change introduced in #8317 had no effect on the generated pom file. We then need to "unscope" apiURL and run `publishLocal` for play-docs before running `test` for documentation. * Add play-rconf on module directory (#8388) Signed-off-by: Thibault Meyer <meyer.thibault@gmail.com> * Upgrade netty (#8389) * use foldLeft instead of /: (#8394) https://github.com/scala/scala/blob/312ddb5d1e3436d7e07acad978cb116e9d023164/src/library/scala/collection/IterableOnce.scala#L465 * Update documentation about SBT usage (#8395) * Use Guice to initialize database connection pools (#8410) Fixes #8402. We now pass the Injector value to the initialization code so that it can use Guice to load a value instead of always using the default constructor. Increases visibility of PooledDatabase.pool field so that we can unit test its value more easily. * Link to Markdown plugin directs to 404 page (#8409) Markdown Support plugin now links to IntelliJ IDEA Markdown help page * TemplateImports cleanup (#8403)
I noticed a couple minor code issues while working on the Gradle integration