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 warnings with Scala 3.4. avoid private[this] #12284

Merged
merged 1 commit into from Feb 21, 2024

Conversation

xuwei-k
Copy link
Contributor

@xuwei-k xuwei-k commented Jan 7, 2024

Pull Request Checklist

Helpful things

Fixes

Purpose

fix warnings. use private instead of private[this] in routes-compiler generate code.

Background Context

References

@ahjohannessen
Copy link

@mkurz Can this be merged?

Copy link
Member

@mkurz mkurz left a comment

Choose a reason for hiding this comment

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

Since all tests pass it seems this doesn't do any harm (and we have quite a lot of tests regarding routing, specially also scripted tests). Probably a leftover from old days.

@mkurz mkurz merged commit 00ffdc0 into playframework:main Feb 21, 2024
25 checks passed
@mkurz
Copy link
Member

mkurz commented Feb 21, 2024

@Mergifyio backport 2.9.x

@mkurz mkurz added this to the 2.9.2 / 3.0.2 milestone Feb 21, 2024
Copy link
Contributor

mergify bot commented Feb 21, 2024

backport 2.9.x

✅ Backports have been created

mkurz added a commit that referenced this pull request Feb 21, 2024
[2.9.x] fix warnings with Scala 3.4. avoid `private[this]` (backport #12284) by @xuwei-k
@ahjohannessen
Copy link

@mkurz Would be nice with a minor release with this :)

@mkurz
Copy link
Member

mkurz commented Feb 22, 2024

@mkurz Would be nice with a minor release with this :)

This will be in the next patch release even, however I don't have a rush just because of this pull request, because we do not officially support non-LTS Scala releases. Just be patient 😉

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.

None yet

3 participants