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

{tools/cli/deployment} の翻訳 #1005

Merged
merged 4 commits into from
Dec 3, 2024
Merged

{tools/cli/deployment} の翻訳 #1005

merged 4 commits into from
Dec 3, 2024

Conversation

darkss-xyzzy
Copy link
Contributor

翻訳・修正

  • 変更内容はCONTRIBUTING.md に記載されたワークフローに従っています

関連Issue

#1003

備考

・用語にゆらぎがないか
・各説明の解釈が誤っていないか
・英語ドキュメントで_server_となっていた箇所は、アンダースコアを使った記法だと、前後に半角スペースを入れないとイタリック体にならないため、*サーバー*と訳しましたので、問題ないか確認お願いします。

Copy link
Collaborator

@lacolaco lacolaco left a comment

Choose a reason for hiding this comment

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

翻訳ありがとうございます!コメントした部分の修正をお願いします 🙇

英語ドキュメントで_server_となっていた箇所は、アンダースコアを使った記法だと、前後に半角スペースを入れないとイタリック体にならないため、サーバーと訳しましたので、問題ないか確認お願いします。

こちらについては問題ありません。対応ありがとうございます。

adev-ja/src/content/tools/cli/deployment.md Outdated Show resolved Hide resolved
adev-ja/src/content/tools/cli/deployment.md Outdated Show resolved Hide resolved
adev-ja/src/content/tools/cli/deployment.md Outdated Show resolved Hide resolved
adev-ja/src/content/tools/cli/deployment.md Outdated Show resolved Hide resolved
adev-ja/src/content/tools/cli/deployment.md Outdated Show resolved Hide resolved
adev-ja/src/content/tools/cli/deployment.md Outdated Show resolved Hide resolved
adev-ja/src/content/tools/cli/deployment.md Outdated Show resolved Hide resolved
adev-ja/src/content/tools/cli/deployment.md Outdated Show resolved Hide resolved
adev-ja/src/content/tools/cli/deployment.md Outdated Show resolved Hide resolved
adev-ja/src/content/tools/cli/deployment.md Outdated Show resolved Hide resolved
@darkss-xyzzy
Copy link
Contributor Author

レビューしていただきありがとうございました。
指摘は反映したのですが、
69行目のSuggested changeと108行目の改行の件に関しては、
こちらからもコメントを入れましたので、
特に確認いただければと思います。

Copy link
Collaborator

@lacolaco lacolaco left a comment

Choose a reason for hiding this comment

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

修正ありがとうございました!

@lacolaco lacolaco merged commit 8733d0e into angular:main Dec 3, 2024
6 checks passed
@lacolaco lacolaco linked an issue Dec 3, 2024 that may be closed by this pull request
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.

{tools/cli/deployment} の翻訳
2 participants