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

Backport "Attempt implicit search for old style implicit parameters in Application matchArgs" #20171

Open
wants to merge 1 commit into
base: lts-3.3
Choose a base branch
from

Conversation

mzuehlke
Copy link

Backports #19737 to the LTS branch.

Fixes #19594

This prevents munit from updating to Scala 3.3.x: scalameta/munit#565

@keynmol
Copy link
Contributor

keynmol commented Apr 12, 2024

Really hope this will be backported - unblocking 3.3 on MUnit in turn unblocks several other libraries that have already moved on to LTS, and in turn unblocks Scala Native 0.5.0 upgrades (the chain is long, yeah)

@WojciechMazur
Copy link
Contributor

There might have been a reason why it was not backported into LTS yet. I've started the Open CB to check how it affects other projects.

@WojciechMazur
Copy link
Contributor

WojciechMazur commented Apr 15, 2024

I've started the Open CB to check how it affects other projects.

We've not found any new regressions in the Open CB related to this PR. The failing projects are either already on Scala 3.Next or were failing already on the LTS branch (last LTS nightly 3.3.4-RC1-bin-20240326-feeddb7-NIGHTLY)

@Gedochao Gedochao requested a review from Kordyjan April 16, 2024 08:18
mzuehlke added a commit to mzuehlke/munit that referenced this pull request Apr 26, 2024
As there isn't an artifact for Scala 3 LTS that includes scala/scala3#20171
I gave it a try with 3.4.2-RC1 that already contains scala/scala3#19737

This PR shows that the only thing that changes is the message returned form the compiler and not changes to munit are needed once Scala 3 LTS with the fix gets released.
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.

4 participants