nodejs-ja: add download directory by tgfjt · Pull Request #906 · nodejs/nodejs.org

tgfjt · 2016-09-22T01:30:11Z

Hi 🐈
This is a translation /download/ into Japanese.

ping @nodejs/nodejs-ja
please review this! 🙇
レビューをお願いします！:bow:

対応したところ

index: /ja/download/
current: /ja/download/current
package-manager: /ja/download/package-manager
releases: /ja/download/releases/

current や LTS は、#900 にあわせました。

訳が怪しい・不安な箇所

display-hint(アンカーにhoverで表示される)の「ダウンロード表示：」
releases の NODE_MODULE_VERSION の説明
package-manager 全体的に… Linux のパッケージをよく分かっていないので怪しい

watilde · 2016-09-22T07:44:45Z

locale/ja/download/package-manager.md

細かいですが

- からも利用できます + から利用できます。

watilde · 2016-09-22T07:51:33Z

locale/ja/download/current.md

ここ、原文は Display downloads for なので ダウンロード だけでも意味が通るかなぁと思ってます。
本当は、 %s をダウンロード みたいにできると良いんですが。

ダウンロードにしました！
そう、本来は、%s をダウンロード にしたいですが出来ないので…

watilde · 2016-09-22T07:56:41Z

locale/ja/download/releases.md

ここ難しいですね。後半部分なんですが、 the ABI (application binary interface) version number of Node.js の説明をしてそうなので、ちょっと直訳に寄せたこういう感じではいかがでしょうか？自信ない＞＜

- これにより、再コンパイルする必要のない、コンパイル済みの C++ アドオンバイナリをロードすることが出来る Node.js のバージョンが分かります。 + このバージョンは、再コンパイルすることなくC++アドオンのバイナリーをロード可能か確認するために使われます。

hiroppy · 2016-09-22T20:01:09Z

locale/ja/download/current.md

Node.js の部分で前後にスペースを入れたほうが統一感がありいいと思います。

- その他のプラットフォームのための Node.js のビルドは、Node.js コミュニティのメンバーによってメンテナンスされています。これらは Node.js のコアチームによってサポートされていません。また、最新の Node.jsのリリースと同じ状態ではないかもしれないことにご注意ください。 + その他のプラットフォームのための Node.js のビルドは、 Node.js コミュニティのメンバーによってメンテナンスされています。これらは Node.js のコアチームによってサポートされていません。また、最新の Node.js のリリースと同じ状態ではないかもしれないことにご注意ください。

そうですね！ index, current 両方直しました〜

marocchino · 2016-09-22T09:08:43Z

locale/ja/download/package-manager.md

実際立ち上げてリンクを確認してみたら切れていると思います。
参考: #857 (diff)

あと、下の題名と合わせて欲しいです。

Debian と Ubuntu ベースの Linux ディストリビューション

題名とあわせる＆リンクチェックしました〜

tgfjt · 2016-09-24T07:37:53Z

レビューありがとうございました！修正しました〜
@watilde @abouthiroppy @marocchino

okuryu · 2016-09-24T08:28:16Z

locale/ja/download/package-manager.md

「公式」と「リポジトリ」の間のスペースは不要じゃないでしょうか。

okuryu · 2016-09-24T08:28:36Z

locale/ja/download/package-manager.md

こちらも「リポジトリ」の後ろのスペースは不要に見えます。

どちらも直しました！

okuryu · 2016-09-24T09:02:34Z

locale/ja/download/package-manager.md

「。」が一つ多いですね。

okuryu · 2016-09-24T09:04:47Z

locale/ja/download/package-manager.md

雑なコピペが… 🙇
。。と一緒に直しました〜

index, current, packagemanager, releases.

okuryu · 2016-09-24T14:37:58Z

looks good. 👍

watilde · 2016-09-25T14:41:09Z

おつかれさまです！
+1

hiroppy · 2016-09-25T14:53:18Z

LGTM 🍻

tgfjt · 2016-09-25T21:24:12Z

🍺

hiroppy added i18n Issues/PRs related to the Website Internationalisation needs WG +1 labels Sep 22, 2016

watilde reviewed Sep 22, 2016

View reviewed changes

hiroppy reviewed Sep 22, 2016

View reviewed changes

marocchino reviewed Sep 22, 2016

View reviewed changes

tgfjt force-pushed the locale/ja_download branch from aab4a6c to 1e79033 Compare September 24, 2016 07:35

okuryu reviewed Sep 24, 2016

View reviewed changes

tgfjt force-pushed the locale/ja_download branch from 1e79033 to f3fd662 Compare September 24, 2016 08:36

okuryu reviewed Sep 24, 2016

View reviewed changes

locale/ja/download/package-manager.md Outdated

Copy link

Contributor

okuryu Sep 24, 2016

Choose a reason for hiding this comment

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

「。」が一つ多いですね。

okuryu reviewed Sep 24, 2016

View reviewed changes

tgfjt force-pushed the locale/ja_download branch from f3fd662 to 4e18f9d Compare September 24, 2016 09:46

nodejs-ja: add download directory

4e18f9d

index, current, packagemanager, releases.

hiroppy merged commit 2cbe7c2 into nodejs:master Sep 25, 2016

tgfjt deleted the locale/ja_download branch September 25, 2016 21:23

marocchino removed needs WG +1 labels Oct 3, 2016

Uh oh!

Comments

Conversation

tgfjt commented Sep 22, 2016

対応したところ

訳が怪しい・不安な箇所

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

marocchino Sep 22, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tgfjt commented Sep 24, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

okuryu commented Sep 24, 2016

Uh oh!

watilde commented Sep 25, 2016

Uh oh!

hiroppy commented Sep 25, 2016

Uh oh!

tgfjt commented Sep 25, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

marocchino Sep 22, 2016 •

edited

Loading

tgfjt commented Sep 24, 2016 •

edited

Loading