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

readme should document Metals setup #565

Open
SethTisue opened this issue Nov 3, 2022 · 1 comment
Open

readme should document Metals setup #565

SethTisue opened this issue Nov 3, 2022 · 1 comment
Labels
help wanted Extra attention is needed

Comments

@SethTisue
Copy link
Member

SethTisue commented Nov 3, 2022

working on #554 here in the Scala Open Source spree in Paris

I merged #564 to increase the default heap size.. hit that in the first 5 min

  • the non-JVM projects may fail (e.g. if node isn't installed), so we should have instructions on how to only work on the JVM build (compat212 is the relevant project)
  • we should recommend working in Scala 2.12 to begin with and let CI do the 2.11 compile
  • Metals will complain about 2.11 being present, that's normal
@SethTisue
Copy link
Member Author

SethTisue commented Nov 3, 2022

It wasn't clear to any of us (me, @rjolly, or @AdrienPI) if it was possible to tell Metals to only run the 2.12 JVM tests and ignore the rest. So Metals was fine for editing the code, but we had to switch to sbt and run compat212/test.

@SethTisue SethTisue added the help wanted Extra attention is needed label Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant