Skip to content
This repository was archived by the owner on Sep 8, 2022. It is now read-only.

be compatible with more Scala versions #33

Merged
merged 1 commit into from
Jul 20, 2015

Conversation

SethTisue
Copy link
Member

see scala/scala#4640 for details.
in that PR (targeted for 2.12.0-M3), we are making a binary
incompatible change to Stream. by avoiding using Stream at all here,
we avoid running afoul of the incompatibility.

Stream is arguably overkill here anyway, Iterator is just fine.
so there's no need to change the code back after we're over
the M2/M3 incompatibility hump.

see scala/scala#4640 for details.
in that PR (targeted for 2.12.0-M3), we are making a binary
incompatible change to Stream. by avoiding using Stream at all here,
we avoid running afoul of the incompatibility.

Stream is arguably overkill here anyway, Iterator is just fine.
so there's no need to change the code back after we're over
the M2/M3 incompatibility hump.
@SethTisue
Copy link
Member Author

review by @lrytz

@lrytz
Copy link
Member

lrytz commented Jul 20, 2015

LGTM!

SethTisue added a commit that referenced this pull request Jul 20, 2015
be compatible with more Scala versions
@SethTisue SethTisue merged commit 2e6dd08 into scala:master Jul 20, 2015
@SethTisue SethTisue deleted the be-compatible-with-pr-4640 branch July 20, 2015 20:10
@lrytz
Copy link
Member

lrytz commented Jul 20, 2015

ups, we forgot to update scalaVersion, now it will release against 2.12.0-M1... :-( You can do a release against M2 locally..

lrytz pushed a commit to lrytz/scala-partest that referenced this pull request May 9, 2018
lrytz pushed a commit to lrytz/scala-partest that referenced this pull request May 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants