Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update contrib #861

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

Update contrib #861

wants to merge 3 commits into from

Conversation

Quafadas
Copy link
Contributor

@Quafadas Quafadas commented Oct 1, 2024

Pure documentation - this was my biggest pain point getting started.

Scala js dom currently targets java 17 for maximum compatibility.

```sh
cs launch sbt --jvm 17
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a coursier-less way to do this? I don't think we should include it if it is not possible.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmmm... not that I know of, although there are many things I don't know :-). I would note, that that I bled some time figuring out the JVM 17 requirement, and then plenty of time futzing with env variables in shells before hitting on this as a QuickStart.

I'm not precious on it, so will bow to superior wisdom - it would have helped me.

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not precious on it, so will bow to superior wisdom - it would have helped me.

I understand that it helps local dev, so how about a small paragraph explaining what cs is with a link to the coursier site?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That sounds quite reasonable... BRB.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants