-
-
Notifications
You must be signed in to change notification settings - Fork 333
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 for advanced/queues.md #1004
Add Japanese translation for advanced/queues.md #1004
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
全体的に良さそうです!
少しコメントしたのでよろしくお願いします。
Overall it looks good!
I have made a few comments and would appreciate your help.
docs/advanced/queues.ja.md
Outdated
@@ -0,0 +1,446 @@ | |||
# Queues | |||
|
|||
Vapor Queues ([vapor/queues](https://github.com/vapor/queues)) は、タスクの責任をサイドワーカーに譲渡することができる、純粋なSwiftのキューシステムです。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
半角と全角の間には半角スペースを入れていただきたいです。
Vapor Queues ([vapor/queues](https://github.com/vapor/queues)) は、タスクの責任をサイドワーカーに譲渡することができる、純粋なSwiftのキューシステムです。 | |
Vapor Queues ([vapor/queues](https://github.com/vapor/queues)) は、タスクの責任をサイドワーカーに譲渡することができる、純粋な Swift のキューシステムです。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@KaitoMuraoka
ありがとうございます。
修正いたしました。Insert spaces between half-width and full-width characters.
docs/advanced/queues.ja.md
Outdated
|
||
このパッケージは [Ruby Sidekiq](https://github.com/mperham/sidekiq) に似ており、以下の機能を提供します: | ||
|
||
- シャットダウン、再起動、または新しいデプロイを示すためにホスティングプロバイダーから送信される`SIGTERM`および`SIGINT`シグナルの安全な処理。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
上記の理由と同様:半角スペース
- シャットダウン、再起動、または新しいデプロイを示すためにホスティングプロバイダーから送信される`SIGTERM`および`SIGINT`シグナルの安全な処理。 | |
- シャットダウン、再起動、または新しいデプロイを示すためにホスティングプロバイダーから送信される `SIGTERM` および `SIGINT` シグナルの安全な処理。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docs/advanced/queues.ja.md
Outdated
try app.queues.use(.redis(url: "redis://127.0.0.1:6379")) | ||
``` | ||
|
||
### Registering a `Job` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
現状のままでも良いのですが、以下のように翻訳しても良いと思います。
### Registering a `Job` | |
### `Job` を登録する(Registering a `Job`) |
簡体字でも同様なのことをしているので
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
タイトルに関しては、やはり英語から変更してしまうと、URLが変わってしまうため変えないで統一したほうが良いと自分は判断いたしました。
Job
を登録する(Registering a Job
)にすると
http://localhost:8000/ja/advanced/queues/#registering-a-job
が
http://localhost:8000/ja/advanced/queues/#job-registering-a-job
に変わってしまいました。英語が混ざった上で訳すと他から参照があった場合にリンク切れを起こしてしまいそうです。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ありがとうございます。であればの意味がよくわからなかったのですが、どうしても日本語化したいとの意思は伝わりましたので、mkdocsオプションを入れてidが連番になるのを防ぎつつ日本語化いたしました。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
これいいですね 👍
ありがとうございます!
…dings to Japanese
@lemo-nade-room |
I translated the "Advanced > Queues" page into Japanese.