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

Scala 2.13.12 (was 2.13.10) #18525

Merged
merged 2 commits into from
Sep 21, 2023
Merged

Scala 2.13.12 (was 2.13.10) #18525

merged 2 commits into from
Sep 21, 2023

Conversation

SethTisue
Copy link
Member

No description provided.

@SethTisue SethTisue marked this pull request as draft September 7, 2023 09:29
@nicolasstucki
Copy link
Contributor

I would add the missing MiMa filter for MapNodeRemoveAllSetNodeIterator.next MiMaFilters.scala StdlibBootstrappedBackwards or StdlibBootstrappedForward. It it seems to be another case where the inferred type is not the same. To solution will be to set this type explicitly in Scala 2.13.

ProblemFilters.exclude[DirectMissingMethodProblem]("scala.collection.immutable.MapNodeRemoveAllSetNodeIterator.next")

@nicolasstucki
Copy link
Contributor

@SethTisue
Copy link
Member Author

@nicolasstucki will do

and while I've got your attention, I just noticed that 3.3.1 ships with 2.13.10 stdlib. this is unfortunate — 2.13.11 came out months ago. will this PR fix it for 3.3.2, if I add the "backport requested" label? @Kordyjan maybe you could add this to the Scala 3 release steps, to check that you aren't shipping an out of date Scala 2 stdlib?

@nicolasstucki
Copy link
Contributor

nicolasstucki commented Sep 11, 2023

2.13.11 came out months ago. will this PR fix it for 3.3.2, if I add the "backport requested" label?

Yes, but we should double-check if changing this version in a patch release if binary-compatible.

I suspect that this would not be a binary-compatible change due to these incompatibilities:

Though these appear when compiling the library with Scala 3, which is not yet fully stable. Therefore these could be irrelevant.

@SethTisue
Copy link
Member Author

We have binary compatibility checks in the scala/scala repo, so I believe that should be considered sufficient assurance. All of the 2.13.x stdlibs are known to be both forward and backward compatible with each other.

@SethTisue SethTisue self-assigned this Sep 20, 2023
SethTisue added a commit to SethTisue/scala that referenced this pull request Sep 20, 2023
context: scala/scala3#18525, which upgrades Scala 3 to use the
2.13.12 stdlib; MiMa there is complaining
SethTisue added a commit to SethTisue/scala that referenced this pull request Sep 20, 2023
context: scala/scala3#18525, which upgrades Scala 3 to use the
2.13.12 stdlib; MiMa there is complaining
SethTisue added a commit to SethTisue/scala that referenced this pull request Sep 20, 2023
context: scala/scala3#18525, which upgrades Scala 3 to use the
2.13.12 stdlib; MiMa there is complaining
@SethTisue
Copy link
Member Author

solution will be to set this type explicitly in Scala 2.13

agree. PR: scala/scala#10549

@SethTisue SethTisue added the backport:nominated If we agree to backport this PR, replace this tag with "backport:accepted", otherwise delete it. label Sep 20, 2023
@SethTisue SethTisue changed the title Scala 2.13.12 Scala 2.13.12 (was 2.13.10) Sep 20, 2023
@SethTisue
Copy link
Member Author

Maybe we can also remove main/project/TastyMiMaFilters.scala#L45-L47

Not yet, I don't think, since we're jumping straight from 2.13.10 to 2.13.12 here. (I've changed the PR title to make that more obvious.)

@SethTisue
Copy link
Member Author

Reviewers: if you're wondering why CompletionSuite.scala needed to be updated, well.... it makes sense that languageFeatures.higherKinds would be skipped by completion, because it's deprecated. But we deprecated it way back in 2.13.1, so I don't know why the change would only be showing up here now. It doesn't seem wondering too hard about, since I think it's a progression.

@SethTisue SethTisue marked this pull request as ready for review September 20, 2023 15:37
@SethTisue
Copy link
Member Author

@nicolasstucki this is ready for you to take a second look at

@nicolasstucki nicolasstucki merged commit 3e716b2 into scala:main Sep 21, 2023
@SethTisue SethTisue deleted the scala-2.13.12 branch September 21, 2023 13:30
@Kordyjan Kordyjan removed the backport:nominated If we agree to backport this PR, replace this tag with "backport:accepted", otherwise delete it. label Oct 10, 2023
@Kordyjan Kordyjan added this to the 3.4.0 milestone Dec 20, 2023
SethTisue added a commit that referenced this pull request Jan 30, 2024
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.

3 participants