Skip to content

i18n(ja): Update github.mdx i18n #10867

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

Merged
merged 9 commits into from
Feb 12, 2025

Conversation

debiru
Copy link
Contributor

@debiru debiru commented Feb 1, 2025

Description (required)

The version of action we are using is out of date and github pages deploy fails, so we updated it.

As of 2025-01-30, the current code fails to deploy to GitHub Pages.

See here for details. https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

Related issues & labels (optional)

Nothing.

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

netlify bot commented Feb 1, 2025

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit c6bae12
🔍 Latest deploy log https://app.netlify.com/sites/astro-docs-2/deploys/67ace1573a239d0008c0cb1f
😎 Deploy Preview https://deploy-preview-10867--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@astrobot-houston
Copy link
Contributor

astrobot-houston commented Feb 1, 2025

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
ja/guides/deploy/github.mdx Localization changed, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@astrobot-houston
Copy link
Contributor

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉

Here’s what will happen next:

  1. Our GitHub bots will run to check your changes.
    If they spot any broken links you will see some error messages on this PR.
    Don’t hesitate to ask any questions if you’re not sure what these mean!

  2. In a few minutes, you’ll be able to see a preview of your changes on Netlify 🥳.

  3. One or more of our maintainers will take a look and may ask you to make changes.
    We try to be responsive, but don’t worry if this takes a few days.

Copy link
Contributor

@morinokami morinokami left a comment

Choose a reason for hiding this comment

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

@debiru こちら対応いただきありがとうございます!

変更内容自体は問題なさそうです。ただ、Astro Docs では各言語版とオリジナルの英語版との差分を https://i18n.docs.astro.build/ にてトラッキングしており、このままこれをマージしてしまうと英語版との差分がないことになってしまうのですが、https://github.com/withastro/docs/commits/main/src/content/docs/en/guides/deploy/github.mdx?since=2024-02-29T14:26:11.000Z を確認するとこれ以外にも積み上がっている修正があるため、すみませんが英語版との差分コミットをすべて反映してもらうことは可能でしょうか?やり方は自由ですが、最終的に作業時点での英語版を漏れなく訳せていることがゴールとなります。

@debiru
Copy link
Contributor Author

debiru commented Feb 2, 2025

@morinokami 承知しましたが、現状の最新英語版は内容に誤りがあるのでそれを修正してからこちらを対応することにします。

@debiru debiru force-pushed the fix/update-guides-deploy-github branch from c3e2937 to 9be0bc1 Compare February 2, 2025 14:23
@debiru
Copy link
Contributor Author

debiru commented Feb 2, 2025

@morinokami

現状の最新英語版は内容に誤りがあるのでそれを修正してからこちらを対応することにします。

これについては、修正方針が不明だったため #10868 で Issue を作成しておきました。

本 PR では、現状の最新英語版 128070a を基にして全面的に再翻訳を行いました。

@debiru debiru requested a review from morinokami February 2, 2025 15:28
Copy link
Contributor

@morinokami morinokami left a comment

Choose a reason for hiding this comment

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

何点かコメントしましたのでご確認をお願いします。


[GitHub Pages](https://pages.github.com/)を使って、[GitHub.com](https://github.com/)上のリポジトリから直接Astroウェブサイトをホストすることができます。

## デプロイ方法

[GitHub Actions](https://github.com/features/actions)を使うことで、AstroサイトをGitHub Pagesに自動的にビルドしてデプロイすることができます。これを行うには、ソースコードがGitHubでホストされている必要があります。

Astroは公式の`withastro/action`を保守しており、ほとんど設定することなくプロジェクトをデプロイすることができます。以下の手順に従って、AstroサイトをGitHubページにデプロイしてください。さらに詳しい情報が必要な場合は、[パッケージのREADME](https://github.com/withastro/action)を参照してください。
Astroは公式の `withastro/action` を保守しており、ほとんど設定することなくプロジェクトをデプロイすることができます。以下の手順に従って、AstroサイトをGitHubページにデプロイしてください。さらに詳しい情報が必要な場合は、[パッケージのREADME](https://github.com/withastro/action)を参照してください。
Copy link
Contributor

Choose a reason for hiding this comment

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

https://github.com/withastro/docs/blob/main/i18n-guides/%E6%97%A5%E6%9C%AC%E8%AA%9E.md に翻訳ガイドがあるのですが、このプロジェクトでは特に英単語等の周りにスペースは挿入しないようにしているため、この修正は不要と思います:

Suggested change
Astroは公式の `withastro/action` を保守しており、ほとんど設定することなくプロジェクトをデプロイすることができます。以下の手順に従って、AstroサイトをGitHubページにデプロイしてください。さらに詳しい情報が必要な場合は、[パッケージのREADME](https://github.com/withastro/action)を参照してください。
Astroは公式の`withastro/action`を保守しており、ほとんど設定することなくプロジェクトをデプロイすることができます。以下の手順に従って、AstroサイトをGitHubページにデプロイしてください。さらに詳しい情報が必要な場合は、[パッケージのREADME](https://github.com/withastro/action)を参照してください。

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.

流儀を知らずすみません。

このファイルのもともとの記述でスペースの有無が揺れていたので、スペース有りに統一してしまっていました。

対応します。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

cd33689 で対応しました。


```js title="astro.config.mjs" ins={4-5}
import { defineConfig } from 'astro/config'
### `github.io` のURLにデプロイする
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
### `github.io` のURLにデプロイする
### `github.io`のURLにデプロイする

Copy link
Contributor Author

Choose a reason for hiding this comment

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

cd33689 で対応しました。

```
- `site` は `https://<YOUR_USERNAME>.github.io` または `https://my-custom-domain.com` である必要があります。
- `base` には、`/my-repo` のように、スラッシュで始まるリポジトリ名を指定します。これは、Astro があなたのウェブサイトのルートがデフォルトの `/` ではなく `my-repo` であることを理解するためです。
`astro.config.mjs` に [`site`](/ja/reference/configuration-reference/#site) と、必要に応じて [`base`](/ja/reference/configuration-reference/#base) オプションを設定します。
Copy link
Contributor

Choose a reason for hiding this comment

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

ここの原文は

Set the site and base options in astro.config.mjs.

ですが、この訳し方だと少し原文から乖離する気がしますが、このように訳した理由はありますか?ここは原文に忠実に、強弱を付けず両者を対等に扱えばいいのではないかと思いました。

Copy link
Contributor Author

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.

d5d2b3a で修正しました。

`site` の値は以下のいずれかでなければなりません。

- あなたのユーザー名に基づいた次のようなURL: `https://<username>.github.io`
- [GitHub Organization's private page](https://docs.github.com/ja/enterprise-cloud@latest/pages/getting-started-with-github-pages/changing-the-visibility-of-your-github-pages-site) 向けに自動生成されたランダムなURL: `https://<random-string>.pages.github.io/`
Copy link
Contributor

Choose a reason for hiding this comment

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

このリンクの中も「GitHub Organizationのプライベートページ」のように訳して問題ない気がしたのですが、ここだけ英語のままとしている理由はありますか?

Copy link
Contributor Author

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.

b6e5f8a で修正しました。


#### `base`

Astroがあなたのリポジトリ名をウェブサイトのルートとして扱うように `base` の値が必要な場合があります。
Copy link
Contributor

Choose a reason for hiding this comment

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

細かいですが、原文の

(e.g. /my-repo)

の部分が欠落してしまっているようです。

Copy link
Contributor Author

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.

8a0f3c5 で修正しました。

- リポジトリが `https://github.com/<USERNAME>/<USERNAME>.github.io` にある場合。
:::

`base` には、`/my-blog` のように、スラッシュで始まるリポジトリ名を指定します。これは、Astroがあなたのウェブサイトのルートがデフォルトの `/` ではなく、`/my-repo` であることを理解するためです。
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
`base` には、`/my-blog` のように、スラッシュで始まるリポジトリ名を指定します。これは、Astroがあなたのウェブサイトのルートがデフォルトの `/` ではなく、`/my-repo` であることを理解するためです
`base`には、`/my-blog`のように、スラッシュで始まるリポジトリ名を指定します。これは、Astroがあなたのウェブサイトのルートがデフォルトの`/`ではなく、`/my-repo`であることを理解できるようにするためです

Copy link
Contributor Author

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.

7f00d40 で修正しました。


2. プロジェクトの `.github/workflows/deploy.yml` に新しいファイルを作成し、以下のYAMLを貼り付けます。
:::tip[カスタムドメインの設定]
以下の `./public/CNAME` ファイルをプロジェクトに追加することで、カスタムドメインを設定することができます。
Copy link
Contributor

Choose a reason for hiding this comment

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

こちらも https://github.com/withastro/docs/blob/main/i18n-guides/%E6%97%A5%E6%9C%AC%E8%AA%9E.md にあるのですが、「することができます」などは表現として冗長なためなるべく使用しないようにしており、ここも不要かと思います:

Suggested change
以下の `./public/CNAME` ファイルをプロジェクトに追加することで、カスタムドメインを設定することができます
以下の`./public/CNAME`ファイルをプロジェクトに追加することで、カスタムドメインを設定できます

Copy link
Contributor Author

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.

fb46a62 で修正しました。

@morinokami
Copy link
Contributor

@debiru こちらコメントしておりますので、まだであればご確認をお願いいたします!

@debiru
Copy link
Contributor Author

debiru commented Feb 12, 2025

@morinokami おっとすみません。今気づきました。もろもろ対応します。

@debiru debiru requested a review from morinokami February 12, 2025 04:03
Copy link
Contributor

@morinokami morinokami left a comment

Choose a reason for hiding this comment

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

関係ない箇所も含め対応いただきありがとうございました、LGTM! 👍

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 job! 🙌

@yanthomasdev yanthomasdev merged commit 3810067 into withastro:main Feb 12, 2025
10 checks passed
@debiru debiru deleted the fix/update-guides-deploy-github branch February 13, 2025 01:48
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.

4 participants