Skip to content

adjust the lastFailure change to be source compatible#327

Merged
SethTisue merged 1 commit intoscala:1.2.xfrom
SethTisue:source-compatible-lastFailure
Nov 30, 2020
Merged

adjust the lastFailure change to be source compatible#327
SethTisue merged 1 commit intoscala:1.2.xfrom
SethTisue:source-compatible-lastFailure

Conversation

@SethTisue
Copy link
Copy Markdown
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
Copy Markdown
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