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

Support >22 args in Scala 3 #747

Merged
merged 5 commits into from
Mar 12, 2024
Merged

Conversation

mkurz
Copy link
Member

@mkurz mkurz commented Mar 8, 2024

Try to fix Fixes

This is WIP.

If this works

  • theTemplate.scala should be moved into a scala-2.13 (also scala-2.12?) folder and
  • in the twirl compiler we can just make use of the ScalaCompat sc parameter to decide wheter we need to add ... with _root_.play.twirl.api.TemplateX[...] or not to stay compatible with Scala 2.13.
  • Also MiMa filters need to be adjusted.

@mkurz mkurz marked this pull request as ready for review March 12, 2024 09:57
@mkurz
Copy link
Member Author

mkurz commented Mar 12, 2024

Fix was quite easy and we can even ship it in a patch release.
Also added tests 👍

@mkurz mkurz merged commit c5fba6a into playframework:main Mar 12, 2024
15 checks passed
@mkurz mkurz deleted the rm-Template0-22 branch March 12, 2024 09:57
@mkurz
Copy link
Member Author

mkurz commented Mar 12, 2024

@Mergifyio backport 1.6.x

Copy link
Contributor

mergify bot commented Mar 12, 2024

backport 1.6.x

✅ Backports have been created

mergify bot added a commit that referenced this pull request Mar 12, 2024
[1.6.x] Support >22 args in Scala 3 (backport #747) by @mkurz
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.

1 participant