Skip to content

adjust the lastFailure change to be source compatible #327

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

Merged
merged 1 commit into from
Nov 30, 2020

Conversation

SethTisue
Copy link
Member

@SethTisue SethTisue commented Nov 21, 2020

sequel to #234

in the community build I saw this compile error:

[playframework] [error] /home/jenkins/workspace/scala-2.13.x-jdk11-integrate-community-build/target-0.9.17/project-builds/playframework-09235f39491e975fec63cc899de593c88cef0cc0/dev-mode/routes-compiler/src/main/scala/play/routes/compiler/RoutesFileParser.scala:167:38: RoutesFileParser.this.Success.type does not take parameters
[playframework] [error]         case Failure(_, _) => Success(elems.toList, in0)
[playframework] [error]                                      ^

in the community build I saw this compile error:

[playframework] [error] /home/jenkins/workspace/scala-2.13.x-jdk11-integrate-community-build/target-0.9.17/project-builds/playframework-09235f39491e975fec63cc899de593c88cef0cc0/dev-mode/routes-compiler/src/main/scala/play/routes/compiler/RoutesFileParser.scala:167:38: RoutesFileParser.this.Success.type does not take parameters
[playframework] [error]         case Failure(_, _) => Success(elems.toList, in0)
[playframework] [error]                                      ^
@SethTisue
Copy link
Member Author

SethTisue commented Nov 21, 2020

I don't completely understand the code involved here, but this is my best guess at what a good fix would be

@Soya-Onishi are you still around to take a look?

@SethTisue SethTisue requested a review from Philippus November 21, 2020 21:58
@SethTisue SethTisue merged commit 14fa7ee into scala:1.2.x Nov 30, 2020
@SethTisue SethTisue deleted the source-compatible-lastFailure branch May 13, 2022 12:36
SethTisue pushed a commit to SethTisue/scala-parser-combinators that referenced this pull request May 13, 2022
SethTisue pushed a commit to SethTisue/scala-parser-combinators that referenced this pull request May 13, 2022
SethTisue pushed a commit to SethTisue/scala-parser-combinators that referenced this pull request May 13, 2022
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