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

Fix unclear SBT example in documentation #44

Merged
merged 2 commits into from
Nov 6, 2020

Conversation

stephenjudkins
Copy link
Contributor

No description provided.

Copy link
Collaborator

@regadas regadas left a comment

Choose a reason for hiding this comment

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

@stephenjudkins Thanks for this!

Technically this is not a typo, this how one would add it as a scala.js dep or cross building. Maybe we could reword this a bit so we could fit both?

@regadas
Copy link
Collaborator

regadas commented Nov 5, 2020

// use this snippet for the JVM
libraryDependencies += "org.typelevel" %% "cats-parse" % "0.0.1"

// use this snippet for JS, or cross-building
libraryDependencies += "org.typelevel" %%% "cats-parse" % "0.0.1"

@stephenjudkins
Copy link
Contributor Author

Yeah, I'm not certain what the right way would be here, but I would personally err on just using %% since that's what most Scala users (who aren't using scala.js) will default to. Your call!

@johnynek
Copy link
Collaborator

johnynek commented Nov 6, 2020

@stephenjudkins do you want to update the PR with @regadas suggestion? I agree it would be helpful for people less familar with these sbt esoterica.

@stephenjudkins stephenjudkins changed the title Fix typo in documentation Fix unclear SBT example in documentation Nov 6, 2020
@johnynek johnynek merged commit d837fcb into typelevel:main Nov 6, 2020
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.

3 participants