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

Updating For Play 2.9.x Series #186

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jmswenski
Copy link

@jmswenski jmswenski commented May 7, 2024

Added support for 2.9.0+ versions of PlayFramework

  • Dropped support for Scala 2.12 in higher Play version enums
  • DefaultVersionedRoutesCompiler uses VersionNumber for version fiddling
  • Added isLowerThan, isHigherThan VersionNumber helpers
  • 2.9.x routes have a renamed library
  • Twirl version bumped to 1.6.0

Added support for 2.9.0+ versions of PlayFramwork

- Dropped support for Scala 2.12 in higher Play version enums
- DefaultVersionedRoutesCompiler uses VersionNumber for version
  fiddling
- Added isLowerThan, isHigherThan VersionNumber helpers
- 2.9.x routes have a renamed library
- Twirl version bumped to 1.6.0
@mkurz
Copy link

mkurz commented May 8, 2024

@jmswenski Are you aware of https://github.com/orgs/playframework/discussions/12338, a new official build-in Gradle Play plugin?

@jmswenski
Copy link
Author

Oh hay! @mkurz No I was not aware of that. Will def start moving in that direction though. Do you think we could still have these changes merged?

We just completed a Java21 upgrade from a 2.8.x base and we're trying to increment our way up to 3.x

@mkurz
Copy link

mkurz commented May 8, 2024

Do you think we could still have these changes merged?

I am not maintaining this repo.

@jmswenski
Copy link
Author

Do you think we could still have these changes merged?

I am not maintaining this repo.

Whoops, my bad - I just figured @mkurz was omnipotent with everything to do with Play ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants