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

add japanese translation (basic part) #7461

Merged
merged 14 commits into from
Dec 17, 2019

Conversation

umi-umi
Copy link
Contributor

@umi-umi umi-umi commented Nov 24, 2019

Description

add japanese translation for document;

docs/ja/
getting_started_introduction.md
cli.md
cli_configuration.md
contributing.md
getting_started_github.md
getting_started_getting_help.md

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

docs/ja/cli.md Outdated Show resolved Hide resolved
docs/ja/cli.md Outdated Show resolved Hide resolved
docs/ja/cli.md Outdated Show resolved Hide resolved
docs/ja/cli.md Outdated Show resolved Hide resolved
docs/ja/cli.md Outdated Show resolved Hide resolved
docs/ja/cli.md Outdated Show resolved Hide resolved
docs/ja/cli.md Outdated Show resolved Hide resolved
docs/ja/cli.md Outdated Show resolved Hide resolved
docs/ja/cli.md Outdated Show resolved Hide resolved
docs/ja/cli.md Outdated Show resolved Hide resolved
@shelaf
Copy link
Contributor

shelaf commented Dec 3, 2019

@umi-umi umiさんのdocs-translations-basicブランチに対してマージを行うと、このpull requestに対して更新が行われるため、本来変更したかった翻訳ドキュメント以外も変更があったように見えてしまい、よく分からなくなってしまいます。
現在のところ、日本語の翻訳ドキュメントを作成している人はumiさん以外いないので、qmkのmasterブランチが更新されても競合が発生する可能性はかなり低いです。docs-translations-basicブランチに対してmasterブランチの更新を取り込む作業は、実際に競合が発生した時に行えば十分です。

@umi-umi
Copy link
Contributor Author

umi-umi commented Dec 3, 2019

docs-translations-basicブランチに対してmasterブランチの更新を取り込む作業は、実際に競合が発生した時に行えば十分です。

ご指摘ありがとうございます。承知しました。

@shelaf
Copy link
Contributor

shelaf commented Dec 3, 2019

@umi-umi pull requestの内容をわかりやすくするため、一度以下のように rebase していただいても良いでしょうか。

umiさんのgithubのqmk_firmwareリポジトリからcloneしてきたqmk_firmwareフォルダにて以下のコマンドを実行します。なお、docs-translations-basicブランチが checkout されている状態とします。また、qmkのgithubのqmk_firmwareリポジトリをupstreamという名前で git remote add しているものとします。別名を付けた場合はその名前に読み替えてください。

git fetch upstream
git rebase upstream/master

これでqmkのgithubのqmk_firmwareリポジトリの更新情報を取得し、docs-translations-basicブランチに加えられた変更履歴がupstream/masterに対して順に変更を加えたかのように履歴が改変されます(このとき、ハッシュ値 f0cac3a でできていたマージコミットはなくなります。git log --oneline --graph --all --decorate 等で確認してください)。そうしたら

git push -f

で、umiさんのgithubのqmk_firmwareリポジトリに対して強制的に(履歴を改変したので git push では無理)変更内容を push してください。そうするとこのpull requestの内容も自動的に更新されて、pull requestを作成した時のように6ファイル分だけの変更のようになるはずです。

@umi-umi umi-umi force-pushed the docs-translations-basic branch from 62ef472 to 0910e2e Compare December 4, 2019 03:26
@umi-umi
Copy link
Contributor Author

umi-umi commented Dec 4, 2019

詳細な手順・説明まで教えていただきありがとうございます!

docs/ja/cli_configuration.md Outdated Show resolved Hide resolved
docs/ja/cli_configuration.md Outdated Show resolved Hide resolved
docs/ja/cli_configuration.md Outdated Show resolved Hide resolved
docs/ja/contributing.md Outdated Show resolved Hide resolved
docs/ja/contributing.md Outdated Show resolved Hide resolved
docs/ja/contributing.md Outdated Show resolved Hide resolved
docs/ja/contributing.md Outdated Show resolved Hide resolved
docs/ja/contributing.md Outdated Show resolved Hide resolved
docs/ja/contributing.md Outdated Show resolved Hide resolved
docs/ja/contributing.md Outdated Show resolved Hide resolved
docs/ja/contributing.md Outdated Show resolved Hide resolved
docs/ja/contributing.md Outdated Show resolved Hide resolved
docs/ja/contributing.md Outdated Show resolved Hide resolved
docs/ja/getting_started_getting_help.md Outdated Show resolved Hide resolved
docs/ja/getting_started_getting_help.md Outdated Show resolved Hide resolved
docs/ja/getting_started_getting_help.md Outdated Show resolved Hide resolved
docs/ja/getting_started_github.md Outdated Show resolved Hide resolved
docs/ja/getting_started_github.md Outdated Show resolved Hide resolved
docs/ja/getting_started_github.md Outdated Show resolved Hide resolved
docs/ja/getting_started_github.md Outdated Show resolved Hide resolved
docs/ja/getting_started_github.md Outdated Show resolved Hide resolved
docs/ja/getting_started_github.md Outdated Show resolved Hide resolved
docs/ja/getting_started_github.md Outdated Show resolved Hide resolved
docs/ja/getting_started_github.md Outdated Show resolved Hide resolved
docs/ja/getting_started_introduction.md Outdated Show resolved Hide resolved
docs/ja/getting_started_introduction.md Outdated Show resolved Hide resolved
docs/ja/getting_started_introduction.md Outdated Show resolved Hide resolved
docs/ja/getting_started_introduction.md Outdated Show resolved Hide resolved
@shelaf
Copy link
Contributor

shelaf commented Dec 13, 2019

@mtei 全てレビューしたので mtei さんが良ければマージしてしまっていいと思います。
#7619 で原文の不備を修正していただいていますが、これだと不完全なので把握している他の原文の不備を含めて別途PRしておきます。

Copy link
Contributor

@mtei mtei left a comment

Choose a reason for hiding this comment

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

レビューが遅くなってすみません、まずは getting_started_introduction.mdを。

docs/ja/getting_started_introduction.md Show resolved Hide resolved
docs/ja/getting_started_introduction.md Outdated Show resolved Hide resolved
docs/ja/getting_started_introduction.md Outdated Show resolved Hide resolved
docs/ja/getting_started_introduction.md Outdated Show resolved Hide resolved
Copy link
Contributor

@mtei mtei left a comment

Choose a reason for hiding this comment

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

contributing.md のレビューです。

docs/ja/contributing.md Show resolved Hide resolved

# どうやって貢献することができますか?

以前にオープンソースに貢献したことはありませんか?QMK で貢献がどのように機能するかが疑問ですか?ここに簡単な説明があります!
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
以前にオープンソースに貢献したことはありませんか?QMK で貢献がどのように機能するかが疑問ですか?ここに簡単な説明があります!
以前にオープンソースに貢献したことはありませんか? QMK で貢献がどのように機能するかが疑問ですか? ここに簡単な説明があります!

Copy link
Contributor

Choose a reason for hiding this comment

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

句読点後のスペース入れていないのでこれはなくていいと思います

Copy link
Contributor

Choose a reason for hiding this comment

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

うーん。私は文末を明示するためにスペース入れたい気分です。
句点の「。」みたいにフォントデザインそのものに、スペースがあるならいらないですが。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

改行無しで ! や ? が出てきた時には、どうしようかといつも悩みます。
(! や ? は見た目的に全角に置き換えたいと思うのですが、そうすると句読点扱いしないツールが多くて、どうしたものかと悩みどころです。。)

今回は、スペースを入れたものに変更してみました。
もし不適切なようであればご指摘ください。

docs/ja/contributing.md Outdated Show resolved Hide resolved
docs/ja/contributing.md Outdated Show resolved Hide resolved
docs/ja/contributing.md Outdated Show resolved Hide resolved

## キーボード

キーボードは QMK の存在理由です。一部のキーボードはコミュニティによって管理されていますが、他のキーボードは特定のキーボードを作成する責任者によって管理されています。`readme.md` を見ると特定のキーボードを管理しているのが誰かが分かります。特定のキーボードに関する質問がある場合、[Issue を開いて](https://github.com/qmk/qmk_firmware/issues)質問にメンテナをタグ付けしてください。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
キーボードは QMK の存在理由です。一部のキーボードはコミュニティによって管理されていますが、他のキーボードは特定のキーボードを作成する責任者によって管理されています`readme.md` を見ると特定のキーボードを管理しているのが誰かが分かります。特定のキーボードに関する質問がある場合、[Issue を開いて](https://github.com/qmk/qmk_firmware/issues)質問にメンテナをタグ付けしてください。
キーボードは QMK の存在理由です。一部のキーボードはコミュニティによって管理されていますが、他のキーボードはそれぞれのキーボードを作成する責任者によって管理されています`readme.md` を見るとそのキーボードを管理しているのが誰かが分かります。特定のキーボードに関する質問がある場合、[Issue を開いて](https://github.com/qmk/qmk_firmware/issues)質問にメンテナをタグ付けしてください。

「メンテナをタグ付けする」原文の tag the maintainer は、具体的な PR の中でよく見かける表現で、最初のうち私はどういうことなのかわかりませんでした。結局、「@誰それ」みたいな記述を書いておいてその人に通知が行くようにすることのようですが、訳注で説明した方が良いような気がしています。shela さん、意見ありませんか?

Copy link
Contributor

Choose a reason for hiding this comment

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

私はどちらでもいいです。ただ、あれはタグというよりメンションだと思ってます。

Copy link
Contributor

Choose a reason for hiding this comment

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

メンションですよねぇ。

GitHub のヘルプでも、
https://help.github.com/ja/github/writing-on-github/basic-writing-and-formatting-syntax#mentioning-people-and-teams

GitHub上の人あるいは Team は、@ に加えてユーザ名もしくは Team 名を入力することでメンションできます。
これによって通知が発生し、会話に注意してもらうことができます。
コメントを編集してユーザ名や Team 名をメンションすれば、人々に通知を受信してもらえます。

と書いてあるし。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

行末に、「(訳注: タグ付け は メンションする という意味です。)」を追記する形で盛り込みました。

docs/ja/cli.md Show resolved Hide resolved
@shelaf
Copy link
Contributor

shelaf commented Dec 15, 2019

原文の不備として #7642 を投げました。一部このプルリクエストに含まれるファイルが存在しているので、#7462 が取り込まれた場合は変更内容を反映してもらえると助かります。

docs/ja/contributing.md Outdated Show resolved Hide resolved
docs/ja/contributing.md Outdated Show resolved Hide resolved
Copy link
Contributor

@mtei mtei left a comment

Choose a reason for hiding this comment

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

Looks good to me.

Copy link
Member

@noroadsleft noroadsleft left a comment

Choose a reason for hiding this comment

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

Thanks!

@noroadsleft noroadsleft merged commit a860da8 into qmk:master Dec 17, 2019
@shelaf shelaf mentioned this pull request Dec 18, 2019
13 tasks
benjaminmikiten added a commit to benjaminmikiten/qmk_firmware that referenced this pull request Dec 18, 2019
* master: (99 commits)
  [Keymap] Added userspace for d4mation. Included their keymap for the Atreus62 (qmk#7483)
  [Keymap] Custom user keymap for Think6.5 with LED range control (qmk#7603)
  [Keymap] CRKBD Custom Keymap - KidBrazil (qmk#7630)
  [Keymap] Add pico 70 keys keymap (qmk#7654)
  Tidy up dztech default keymaps and info.json (qmk#7608)
  Heisenberg handwired keyboard added (qmk#7643)
  [Keyboard] Added Filco Majestouch TKL Pegasus Hoof ISO Layout (qmk#7647)
  Ported J80 to QMK (qmk#7488)
  [Keyboard] Magnavox Videowriter conversion with Pro Micro (qmk#7634)
  [Docs] add japanese translation (basic part) (qmk#7461)
  Tidy up dztech rules.mk
  Relocate RGB keycode processing (qmk#7508)
  Move kwerdenker's personal keymap from RGB (qmk#7645)
  Remove QMK_KEYBOARD_CONFIG_H from boards (qmk#7635)
  Disable usb on slave half to resolve random 'lockup' (qmk#7649)
  [Core] Optimize matrix processing (qmk#7621)
  [Keymap] boy_314's satisfaction75 layout (qmk#7638)
  [Keyboard] XD68 65% ATMega32U4 based (qmk#7395)
  [keyboard] Plain60 cleanups (qmk#7644)
  update default h88 keymap (qmk#7646)
  ...
patrl pushed a commit to patrl/qmk_firmware that referenced this pull request Dec 29, 2019
* add basic part

* update cli

* update cli

* correct links to ja document

* update files based on comments

* update for a postpositional particle

* update for a punctuation

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

Co-Authored-By: shela <shelaf@users.noreply.github.com>
Co-Authored-By: Takeshi ISHII <2170248+mtei@users.noreply.github.com>
HokieGeek pushed a commit to HokieGeek/qmk_firmware that referenced this pull request Feb 21, 2020
* add basic part

* update cli

* update cli

* correct links to ja document

* update files based on comments

* update for a postpositional particle

* update for a punctuation

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

Co-Authored-By: shela <shelaf@users.noreply.github.com>
Co-Authored-By: Takeshi ISHII <2170248+mtei@users.noreply.github.com>
kylekuj pushed a commit to kylekuj/qmk_firmware that referenced this pull request Apr 21, 2020
* add basic part

* update cli

* update cli

* correct links to ja document

* update files based on comments

* update for a postpositional particle

* update for a punctuation

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

Co-Authored-By: shela <shelaf@users.noreply.github.com>
Co-Authored-By: Takeshi ISHII <2170248+mtei@users.noreply.github.com>
@umi-umi umi-umi deleted the docs-translations-basic branch May 3, 2020 03:42
BorisTestov pushed a commit to BorisTestov/qmk_firmware that referenced this pull request May 23, 2024
* add basic part

* update cli

* update cli

* correct links to ja document

* update files based on comments

* update for a postpositional particle

* update for a punctuation

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

* update files based on comments

Co-Authored-By: shela <shelaf@users.noreply.github.com>
Co-Authored-By: Takeshi ISHII <2170248+mtei@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants