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

Exclude sourcecode from scalameta parsers #3195

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

nafg
Copy link
Contributor

@nafg nafg commented Oct 31, 2024

Because it pulls in sourcecode for Scala 2.13 even on Scala 3, this just wasted a lot of my time trying to figure out why I was getting bizarre errors caused by Scala 3 not being able to run Scala 2 macros.

@nafg nafg closed this Oct 31, 2024
@nafg
Copy link
Contributor Author

nafg commented Oct 31, 2024

Oh. Production code depends on other parts of scalameta. Perhaps I should add another exclusion then.

@nafg nafg reopened this Oct 31, 2024
build.sbt Outdated Show resolved Hide resolved
@nafg nafg changed the title Make scalameta parsers a test dependency Exclude sourcecode from scalameta parsers Oct 31, 2024
build.sbt Outdated Show resolved Hide resolved
@nafg
Copy link
Contributor Author

nafg commented Oct 31, 2024

I don't think the remaining failures have to do with this change

@nafg
Copy link
Contributor Author

nafg commented Oct 31, 2024

The commits should probably be squashed

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