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

i18n(ja): Update integrations-guide.mdx #7014

Merged

Conversation

morinokami
Copy link
Contributor

@morinokami morinokami commented Feb 24, 2024

Copy link

vercel bot commented Feb 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview Mar 7, 2024 4:36pm

@github-actions github-actions bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Feb 24, 2024
Copy link
Contributor

@SnowDingo SnowDingo left a comment

Choose a reason for hiding this comment

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

一つ改善点はありますが、問題なさそうです。
長文翻訳ありがとうございます!

Copy link
Contributor

@SnowDingo SnowDingo left a comment

Choose a reason for hiding this comment

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

下にレビューした時に気になった点をまとめて置いたのでご確認ください。


パッケージマネージャーを使用してnpmパッケージのインテグレーションをインストールし、`astro.config.mjs`を手動で更新します。

たとえば、`@astrojs/sitemap`インテグレーションをインストールするには以下のようにします。
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.

@SnowDingo なるほど、ありがとうございます!こういった漢字の「閉じ・開き」(「全く」と「まったく」、「是非」と「ぜひ」、「又は」と「または」、などなど。https://note.com/swwwitch/n/n105a095c0687 が詳しいです)に関して、これまでは特に日本語訳プロジェクトにおいては厳密なルールなしに個々人の裁量に任せてきた感があるのですが、ここを漢字にしたほうがいい理由などはありますかね?個人的には、特にそうすることで誤読の可能性もないため、読みやすさのためにこの語は平仮名で書くことを通常としていました。

Copy link
Contributor

@SnowDingo SnowDingo Feb 29, 2024

Choose a reason for hiding this comment

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

漢字に直すよりひらがなの方がここは確かに良さそうですね。
僕的には漢字にして書いた方が読みやすいと思ったものの、ひらがなで書いても特に読みやすさが悪化する事は無さそうなので、変更なしでOkだと思います。
ただこういう所は今後さらに出てくるかもしれないので、近いうちに日本語翻訳のルール決めをする必要があると思います。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@SnowDingo そうですね、なかなかすべてのルールを決めたり守ったりするのは大変そうですが、こういうふうに迷わないように指針があると良さそうですね!フィードバックありがとうございます🙏(そして問題なさそうであれば Approve して LGTM を残しておいていただけると)

@morinokami morinokami requested a review from SnowDingo March 1, 2024 12:51
Copy link
Member

@yanthomasdev yanthomasdev left a comment

Choose a reason for hiding this comment

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

LGTM, good work. Thanks!

@yanthomasdev yanthomasdev merged commit a747d92 into withastro:main Mar 7, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants