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

support Scala 3 style wildcard import in RemoveUnused #1664

Merged
merged 1 commit into from
Sep 10, 2022

Conversation

xuwei-k
Copy link
Contributor

@xuwei-k xuwei-k commented Sep 5, 2022

@xuwei-k xuwei-k force-pushed the issue-1663 branch 2 times, most recently from ea505d6 to 93666a4 Compare September 5, 2022 07:14
@xuwei-k xuwei-k marked this pull request as ready for review September 5, 2022 07:45
Copy link
Collaborator

@bjaglin bjaglin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

I wonder if this inconsistency is something we might want to fix upstream, in scalameta or in the compiter @tgodzik?

If not, I think we could leverage the testkit for such tests, by adding a inputSource3 sbt module (a bit like in organize-imports), and put input/output sources in scala-2.12+. I can do that as a follow-up PR @xuwei-k if you don't have time/interest to try that out.

@bjaglin bjaglin merged commit 36e89d2 into scalacenter:main Sep 10, 2022
@bjaglin
Copy link
Collaborator

bjaglin commented Sep 10, 2022

I can do that as a follow-up PR @xuwei-k if you don't have time/interest to try that out.

#1666

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.

RemoveUnused does not clean Scala 3 style wildcard import with -Xsource:3
2 participants