Skip to content

Change access modifier to package private for RoutingDsl.{Route,RouteParam}#13003

Merged
mkurz merged 1 commit into
playframework:mainfrom
WojciechMazur:fix/routing-dsl-access
Dec 4, 2024
Merged

Change access modifier to package private for RoutingDsl.{Route,RouteParam}#13003
mkurz merged 1 commit into
playframework:mainfrom
WojciechMazur:fix/routing-dsl-access

Conversation

@WojciechMazur
Copy link
Copy Markdown
Contributor

The static private modifier for these 2 classes was incorrect. In the previous versions of Scala it compiled due to existence of a bug in the compiler.
The mentioned bug was minimized and discussed in scala/scala3#21599 and was discovered after merging a fix scala/scala3#21362
Problem was discovered by Scala 3 Open Community Build in:

Pull Request Checklist

…Param} - fixes illegal access prohibited since Scala 3.3.5 LTS and 3.6
Copy link
Copy Markdown
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.

Thanks!

@mkurz mkurz merged commit 28447a5 into playframework:main Dec 4, 2024
@mkurz
Copy link
Copy Markdown
Member

mkurz commented Dec 4, 2024

@Mergifyio backport 3.0.x 2.9.x

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Dec 4, 2024

@WojciechMazur WojciechMazur deleted the fix/routing-dsl-access branch December 4, 2024 15:01
mergify Bot added a commit that referenced this pull request Dec 4, 2024
[2.9.x] Change access modifier to package private for  `RoutingDsl.{Route,RouteParam}` (backport #13003) by @WojciechMazur
mergify Bot added a commit that referenced this pull request Dec 4, 2024
[3.0.x] Change access modifier to package private for  `RoutingDsl.{Route,RouteParam}` (backport #13003) by @WojciechMazur
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