Skip to content

Fix typo in README's crossProject example #41

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

Merged
merged 2 commits into from
Dec 5, 2017
Merged

Conversation

Grogs
Copy link
Contributor

@Grogs Grogs commented Dec 2, 2017

No description provided.

@Grogs Grogs changed the title Fix typo in crossProject example Fix typo in README's crossProject example Dec 2, 2017
Copy link
Member

@sjrd sjrd left a comment

Choose a reason for hiding this comment

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

Thanks. There's a small thing that can be improved.

README.md Outdated
@@ -19,7 +19,7 @@ If you have a `crossProject`, the setting must be used only in the JS part:
```scala
lazy val myCross = crossProject.
...
jsSettings.(
.jsSettings(
Copy link
Member

Choose a reason for hiding this comment

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

While you're here, and since it's relevant, you should also remove the . at the end of crossProject.. Otherwise the fixed example still doesn't compile (... notwithstanding).

sbt used to recommend the trailing-dot style in their docs, but I see that now they use the leading-dot style instead (which IMO is better).

Per code review feedback
Copy link
Member

@sjrd sjrd left a comment

Choose a reason for hiding this comment

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

Thanks :)

@sjrd sjrd merged commit d58708a into scala-js:master Dec 5, 2017
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.

2 participants