-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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): Add publish-to-npm.mdx #11048
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for astro-docs-2 ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
e2b8b42
to
34282cf
Compare
## パッケージの作成 | ||
|
||
:::note[必要知識] | ||
この章に入る前に、以下の基本的なことを理解しておくとよいでしょう: |
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.
Before diving in, it will help to have a basic understanding of:
が原文だったので、 しておいてください
だと強いニュアンスになりすぎるかなと思い 理解しておくとよいでしょう
のままで最後を 。
に変えるだけにしました。
やはり、理解しておいてください。
の方が良さそうであればお知らせください 🙏
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.
理解しておくとよいでしょう
で問題ないと思います。
ご確認ありがとうございます。🙏
|
||
この例では、`my-project`という名前のプロジェクトに、`my-component`という単一のパッケージと、コンポーネントのテストとデモ用の`demo/`ディレクトリを作成します。 | ||
|
||
これはプロジェクトルートの`package.json`ファイルで設定されます: |
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.
細かいですが、句読点にしてください。
これはプロジェクトルートの package.json
ファイルで設定されます:
↓
これはプロジェクトルートの package.json
ファイルで設定されます。
|
||
Astroは現在、テストランナーを提供していません。_(この機能の開発に協力していただける方は、[Discordに参加してください!](https://astro.build/chat))_ | ||
|
||
当面のテストに関する推奨事項は以下の通りです: |
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.
細かいですが、句読点にしてください。
当面のテストに関する推奨事項は以下の通りです:
↓
当面のテストに関する推奨事項は以下の通りです。
|
||
コンポーネントに関連するキーワードの配列で、他のユーザーが[npmでコンポーネントを見つける](https://www.npmjs.com/search?q=keywords:astro-component,withastro)のに役立ち、その他の検索カタログでも使用されます。 | ||
|
||
Astroエコシステムでの発見可能性を最大限に高めるために、特別なキーワードとして`astro-component`または`withastro`を追加してください。 |
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.
発見可能性
に違和感を感じたので 発見性
にしました。
Astroエコシステムでの発見可能性を最大限に高めるために、特別なキーワードとしてastro-component
またはwithastro
を追加してください。
↓
Astroエコシステムでの発見性を最大限に高めるために、特別なキーワードとしてastro-component
またはwithastro
を追加してください。
|
||
## パッケージの開発 | ||
|
||
Astroには開発専用の「パッケージモード」はありません。代わりに、プロジェクト内でパッケージを開発およびテストするためのデモプロジェクトを使用する必要があります。これは開発専用のプライベートウェブサイトでも、パッケージのための公開デモ/ドキュメントウェブサイトでも構いません。 |
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.
細かいですが ウェブ
→ web
の方がいいのかなと感じました。
Astroには開発専用の「パッケージモード」はありません。代わりに、プロジェクト内でパッケージを開発およびテストするためのデモプロジェクトを使用する必要があります。これは開発専用のプライベートウェブサイトでも、パッケージのための公開デモ/ドキュメントウェブサイトでも構いません。
↓
Astroには開発専用の「パッケージモード」はありません。代わりに、プロジェクト内でパッケージを開発およびテストするためのデモプロジェクトを使用する必要があります。これは開発専用のプライベートwebサイトでも、パッケージのための公開デモ/ドキュメントwebサイトでも構いません。
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.
後学のために教えていただきたいのですが、ウェブ
を web
と書いたほうがよいという根拠は何でしょうか?
既存の翻訳文言を調べると「ウェブ」が最も多く、次に「Web」が多いようです。「web」と日本語で翻訳している例は少ないようですが、本当に web でよいのでしょうか。
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.
@debiru
ドキュメントウェブサイト
よりも ドキュメントwebサイト
の方が可読性が高いのかな?と感じたため指摘させていただきました。
小文字、大文字は「Web」が多いようでしたら「Web」でいいと思います。
仮の例で適当なワードを上げますが、
Private Website Package Installation Documentation
というテキストがあるとして、
プライベートウェブサイトパッケージインストールドキュメント
と
プライベートWebサイトパッケージのインストールドキュメント
はどちらが読みやすいか?という具合でしょうか、、、?
特に不要そうでしたらコチラの内容はスルーしていただいて大丈夫です。🙏
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.
こういう系はサイト内で用法が統一されていれば良いのかな?と考えて調べてみました。
website
の翻訳としては、現状 ウェブサイト
の方がマジョリティーっぽかったのでそちらを採用するのでも良いかなと考えたのですがどうでしょうか? 👀
1f6ab3c
Header | Header |
---|---|
![]() |
![]() |
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.
であればWebではなくウェブで大丈夫だと思います!
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.
@t0yohei
確認いたしました。
細かいですが諸々コメントを記載しましたのでご確認お願いいたします。
|
||
3. 各ページには、テストしたいコンポーネントの使用例を含めます。 | ||
|
||
4. `astro build`を実行してfixturesをビルドし、`dist/__fixtures__/`ディレクトリの出力を期待する結果と比較します。 |
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.
細かいですが、下記の方が読みやすく感じました。
astro build
を実行してfixturesをビルドし、dist/__fixtures__/
ディレクトリの出力を期待する結果と比較します。
↓astro build
を実行してfixtures
をビルドし、dist/__fixtures__/
ディレクトリの出力を期待する結果と比較します。
|
||
パッケージの準備ができたら、`npm publish`コマンドを使用してnpmに公開できます。失敗した場合は、`npm login`でログインしているか、`package.json`が正しいかを確認してください。成功すれば完了です! | ||
|
||
Astroパッケージには`build`ステップが不要であることに注目してください。`.astro`、`.ts`、`.jsx`、`.css`などのAstroがネイティブにサポートするファイルタイプは、ビルドステップなしで直接公開できます。 |
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.
変なところで改行してしまっているので、いっその事先に改行させていいのかなと思いました。
Astroパッケージにはbuild
ステップが不要であることに注目してください。.astro
、.ts
、.jsx
、.css
などのAstroがネイティブにサポートするファイルタイプは、ビルドステップなしで直接公開できます。
↓
Astroパッケージにはbuild
ステップが不要であることに注目してください。
.astro
、.ts
、.jsx
、.css
などのAstroがネイティブにサポートするファイルタイプは、ビルドステップなしで直接公開できます。
あなたの成果を[インテグレーションライブラリ](https://astro.build/integrations/)に追加して、共有しましょう! | ||
|
||
:::tip | ||
インテグレーションの構築に助けが必要ですか?あるいは他のインテグレーション開発者と交流したいですか?[Discordサーバー](https://astro.build/chat)に専用の`#integrations`チャンネルがあります。ぜひ挨拶に来てください! |
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.
違和感を感じたので微調整させていただきました。
インテグレーションの構築に助けが必要ですか?あるいは他のインテグレーション開発者と交流したいですか?Discordサーバーに専用の#integrations
チャンネルがあります。ぜひ挨拶に来てください!
↓
インテグレーションの構築にサポートが必要ですか?あるいは他のインテグレーション開発者と交流したいですか?Discordサーバーに専用の#integrations
チャンネルがあります。ぜひお越しください!
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.
この修正を簡単に承認できるように変更点を追加しておきました。
|
||
インテグレーションライブラリは、`package.json`から`name`、`description`、`repository`、`homepage`のデータを読み取ります。 | ||
|
||
アバターはライブラリであなたのブランドを際立たせる素晴らしい方法です!パッケージを公開したら、アバターを添付して[GitHubのイシューを作成](https://github.com/withastro/astro.build/issues/new/choose)してください。リストに追加します。 |
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.
調整しました。
アバターはライブラリであなたのブランドを際立たせる素晴らしい方法です!パッケージを公開したら、アバターを添付してGitHubのイシューを作成してください。リストに追加します。
↓
アバターは、ライブラリ内でブランドをアピールするのに最適です!パッケージを公開したら、アバターを添付GitHubのイシューを作成してください。リストに追加します。
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.
すいません、コチラの内容読み返すと日本語がおかしくなっていましたね、、、
下記の内容で良いと思います。
アバターはライブラリであなたのブランドを際立たせる素晴らしい方法です!パッケージを公開したら、アバターを添付してGitHubのイシューを作成してください。リストに追加します。
↓↓↓↓
アバターは、ライブラリ内でブランドをアピールするのに最適です!パッケージを公開したら、アバターを添付し、GitHubのイシューを作成してください。リストに追加します。
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.
8d278ea#diff-f75db2aebd5d7156b67b1fddb13180a8b910395e5faac0d9cefcccfc6d0fdc73R274
アバターは、ライブラリ内でブランドをアピールするのに最適です!パッケージを公開したら、アバターを添付してGitHubのイシューを作成してください。リストに追加します。
としております。そちらだと微妙そうでしょうか? 👀
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.
@t0yohei
問題ないです!
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.
@t0yohei さん、翻訳ありがとうございます。
変更点は既に @n-tong009 さんがほぼ出していたのであまりないのですが、いくつか細かい所があるのでそこは確認していただけると幸いです。
よろしくお願いします。
|
||
## クイックスタート | ||
|
||
コンポーネントの開発をすぐに始めるには、すでにセットアップされたテンプレートを使用できます。 |
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.
細かくて申し訳ないのですが、ここの文章は順番を入れ替えた方が良いと思います。
コンポーネントの開発をすぐに始めるには、すでにセットアップされたテンプレートを使用できます。 | |
すでにセットアップされたテンプレートを使用することでコンポーネントの開発をすぐに始められます。 |
あなたの成果を[インテグレーションライブラリ](https://astro.build/integrations/)に追加して、共有しましょう! | ||
|
||
:::tip | ||
インテグレーションの構築に助けが必要ですか?あるいは他のインテグレーション開発者と交流したいですか?[Discordサーバー](https://astro.build/chat)に専用の`#integrations`チャンネルがあります。ぜひ挨拶に来てください! |
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.
この修正を簡単に承認できるように変更点を追加しておきました。
あなたの成果を[インテグレーションライブラリ](https://astro.build/integrations/)に追加して、共有しましょう! | ||
|
||
:::tip | ||
インテグレーションの構築に助けが必要ですか?あるいは他のインテグレーション開発者と交流したいですか?[Discordサーバー](https://astro.build/chat)に専用の`#integrations`チャンネルがあります。ぜひ挨拶に来てください! |
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.
インテグレーションの構築に助けが必要ですか?あるいは他のインテグレーション開発者と交流したいですか?[Discordサーバー](https://astro.build/chat)に専用の`#integrations`チャンネルがあります。ぜひ挨拶に来てください! | |
インテグレーションの構築にサポートが必要ですか?あるいは他のインテグレーション開発者と交流したいですか?Discordサーバーに専用の#integrationsチャンネルがあります。ぜひお越しください |
アバターはライブラリであなたのブランドを際立たせる素晴らしい方法です!パッケージを公開したら、アバターを添付して[GitHubのイシューを作成](https://github.com/withastro/astro.build/issues/new/choose)してください。リストに追加します。 | ||
|
||
:::tip | ||
NPMから読み取った情報を上書きする必要がありますか?問題ありません!更新された情報を添えて[イシューを作成](https://github.com/withastro/astro.build/issues/new/choose)してください。カスタムの`name`、`description`、`homepage`が使用されるようにします。 |
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.
細かいですが一応ここの言葉選びは少し以下のように直した方が良いと思います。
NPMから読み取った情報を上書きする必要がありますか?問題ありません!更新された情報を添えて[イシューを作成](https://github.com/withastro/astro.build/issues/new/choose)してください。カスタムの`name`、`description`、`homepage`が使用されるようにします。 | |
NPMから取得した情報を上書きする必要がありますか?問題ありません!更新後の情報を添えて[イシューを作成](https://github.com/withastro/astro.build/issues/new/choose)してください。カスタムの`name`、`description`、`homepage`が適応されるようにします。 |
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.
その提案について疑問があります。
Need to override the information our library reads from NPM? No problem! File an issue with the updated information and we’ll make sure the custom name, description, or homepage is used instead.
- reads を「読み取った」から「取得した」に変更した方がよいのでしょうか。
- updated information を「更新された情報」から「更新後の情報」に変更した方がよいのでしょうか。
- is used instead と used が使われていますが「使用」ではなく「適応」という語を選んだ理由は何でしょうか。
@t0yohei さんの翻訳に対しても指摘があります。
NPMから読み取った情報を上書きする必要がありますか?問題ありません!更新された情報を添えてイシューを作成してください。カスタムの
name
、description
、homepage
が使用されるようにします。
- 「NPMから読み取った情報」とありますが、元の文章は reads from NPM ですので、ここでは過去形ではなく現在形「NPMから読み取る情報」の方が原文のままになるかと思います。
- 「が使用されるようにします」とありますが、日本語として不自然です。instead も訳されていません。「が代わりに使われるようになります」はいかがでしょうか。
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.
ありがとうございます、 @debiru さんが提案して下さった翻訳が一番良さそうに思ったので採用いたしました! 🙏
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.
@debiru
確かにこの変更は不必要だったかもしれないですね。無駄な変更を発見していただきありがとうございます。
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.
目に入ったので私もレビューしてみました。細かいですが数点コメントをつけました。お手数ですがご確認をお願いいたします。
import { Steps } from '@astrojs/starlight/components'; | ||
import { FileTree } from '@astrojs/starlight/components'; | ||
|
||
新しいAstroコンポーネントを作成していますか? **[npm!](https://npmjs.com/)に公開しましょう!** |
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.
原文が悪いのですが、リンクのラベルはここでは "npm!" ではなく "npm" にしたほうがよいかと思います。
インテグレーションの構築にサポートが必要ですか?あるいは他のインテグレーション開発者と交流したいですか?[Discordサーバー](https://astro.build/chat)に専用の`#integrations`チャンネルがあります。ぜひお越しください! | ||
::: | ||
|
||
### `package.json` データ |
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.
`package.json` データ
英語と日本語の間にスペースがあるようです。
日本語翻訳ガイド:https://github.com/withastro/docs/blob/main/i18n-guides/%E6%97%A5%E6%9C%AC%E8%AA%9E.md
1ee619e
to
8d278ea
Compare
@debiru @n-tong009 @SnowDingo |
Description (required)
Add publish-to-npm.mdx in Japanese.
Related issues & labels (optional)