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

Fix #971 – "Exception occurred while executing macro expansion." #994

Conversation

sgodbillon
Copy link
Contributor

@sgodbillon sgodbillon commented Mar 3, 2024

Pull Request Checklist

  • Have you read through the contributor guidelines?
  • Have you squashed your commits?
  • Have you added copyright headers to new files?
  • Have you updated the documentation?
  • Have you added tests for any changed functionality?

Fixes

Fixes #971 – "Exception occurred while executing macro expansion."

Purpose

What does this PR do?

Background Context

When the case class has generic parameters, generated methods for default parameters ($lessinit$greater$default$XXX) have the same generic parameters – so it is necessary to provide the type parameters when calling them.

References

Are there any relevant issues / PRs / mailing lists discussions?

@sgodbillon sgodbillon force-pushed the sgodbillon/971-Exception_occurred_while_executing_macro_expansion branch from 9b3b4d7 to ead441a Compare March 4, 2024 22:11
@cchantep cchantep enabled auto-merge (squash) March 5, 2024 13:17
@cchantep cchantep merged commit fe39d65 into playframework:main Mar 5, 2024
14 checks passed
@mkurz
Copy link
Member

mkurz commented Mar 7, 2024

Should this be backported to 2.10.x and be part of a 2.10.5 release?

@mkurz
Copy link
Member

mkurz commented Apr 29, 2024

@Mergifyio backport 2.10.x

Copy link
Contributor

mergify bot commented Apr 29, 2024

backport 2.10.x

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Apr 29, 2024
mkurz added a commit that referenced this pull request Apr 29, 2024
[2.10.x] Fix #971 – "Exception occurred while executing macro expansion." (backport #994) by @sgodbillon
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.

"Exception occurred while executing macro expansion."
3 participants