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

Translate 01_install.md #14

Merged
merged 2 commits into from
May 23, 2015
Merged

Conversation

edvakf
Copy link
Contributor

@edvakf edvakf commented May 20, 2015

refs #7


## Scala

Scala requires JDK for running on the JVM. Please make sure to have it installed.
Copy link
Member

Choose a reason for hiding this comment

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

Scala requires JDK for running on the JVM. は私の感覚だと違和感あります。You need to previously install JDK because Scala runs on the JVM. とかどうでしょう?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

preiouslyは違う気がしますがそんな感じにしてみます。この2文を合わせて Make sure to have the JDK installed because Scala runs on the JVM. で良いでしょうか?

Copy link
Member

Choose a reason for hiding this comment

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

確かに元の文章とは少し外れますね。はい、そちらでお願いします。

seratch added a commit that referenced this pull request May 23, 2015
@seratch seratch merged commit 1c10792 into scalikejdbc:master May 23, 2015
@edvakf edvakf deleted the feature/en_01_install branch May 23, 2015 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants