Skip to content

Conversation

k94-ishi
Copy link
Contributor

This PR is a translation of docs/ja/docs/environment-variables.md. I'm Japanese.

In the original text of this page, parts that seemed more suitable as "info" were labeled as "tip."
For now, I have left them as "tip". (not translate into "豆知識")

@github-actions github-actions bot added the lang-all Translations label Jan 18, 2025
Copy link
Contributor

@alejsdev alejsdev changed the title Add Japanese translation for docs/ja/docs/environment-variables.md 🌐 Add Japanese translation for docs/ja/docs/environment-variables.md Jan 18, 2025
@alejsdev alejsdev added lang-ja Japanese translations awaiting-review labels Jan 18, 2025
Copy link

@kiharito kiharito left a comment

Choose a reason for hiding this comment

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

@k94-ishi
翻訳ありがとうございます!
いくつか気になったところについてコメントしておきました🙏

Comment on lines 212 to 214

## PythonのインストールとPATH環境変数の更新

Choose a reason for hiding this comment

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

目次構成は原文と揃うようにしておく方が良いかなと思います

Suggested change
## PythonのインストールとPATH環境変数の更新
### PythonのインストールとPATH環境変数の更新
スクリーンショット 2025-01-25 21 54 57 スクリーンショット 2025-01-25 21 55 10

Comment on lines 248 to 251

#### 実行例:

ターミナルで以下のコマンドを入力すると:

Choose a reason for hiding this comment

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

ここも原文にない見出しが足されることで目次構成が変わってしまうのが気になりました。

ここは、前にある「This way, when you type python in the terminal, ...」を受けての「So, if you type: (python)」で、両者似たようなことを言っているので、自分ならこんな感じにしそうです

Suggested change
#### 実行例:
ターミナルで以下のコマンドを入力すると:
つまり、ターミナルで以下のコマンドを入力すると:


Pythonをインストールして、そのプログラムが`/opt/custompython/bin`というディレクトリに配置されたとします。

もし、`PATH`環境変数を更新を更新するように答えると、`PATH`環境変数に`/opt/custompython/bin`が追加されます。

Choose a reason for hiding this comment

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

おそらくこうかなと思います

Suggested change
もし、`PATH`環境変数を更新を更新するように答えると`PATH`環境変数に`/opt/custompython/bin`が追加されます。
もし、`PATH`環境変数を更新するように答えると`PATH`環境変数に`/opt/custompython/bin`が追加されます。


Pythonをインストールして、そのプログラムが`C:\opt\custompython\bin`というディレクトリに配置されたとします。

もし、`PATH`環境変数を更新を更新するように答えると、`PATH`環境変数に`C:\opt\custompython\bin`が追加されます。

Choose a reason for hiding this comment

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

ここも同じくです

Suggested change
もし、`PATH`環境変数を更新を更新するように答えると`PATH`環境変数に`C:\opt\custompython\bin`が追加されます。
もし、`PATH`環境変数を更新するように答えると`PATH`環境変数に`C:\opt\custompython\bin`が追加されます。

/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/custompython/bin
```

このようにして、ターミナルで`python`と入力するしたときに、OSは`/opt/custompython/bin`(リストの末尾のディレクトリ)にあるPythonプログラムを見つけ、使用します。

Choose a reason for hiding this comment

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

おそらくこんな感じかなと思います

Suggested change
このようにして、ターミナルで`python`と入力するしたときに、OSは`/opt/custompython/bin`(リストの末尾のディレクトリ)にあるPythonプログラムを見つけ、使用します。
このようにして、ターミナルで`python`と入力したときに、OSは`/opt/custompython/bin`(リストの末尾のディレクトリ)にあるPythonプログラムを見つけ、使用します。

C:\Program Files\Python312\Scripts;C:\Program Files\Python312;C:\Windows\System32;C:\opt\custompython\bin
```

このようにして、ターミナルで`python`と入力するしたときに、OSは`C:\opt\custompython\bin\python`(リストの末尾のディレクトリ)にあるPythonプログラムを見つけ、使用します。

Choose a reason for hiding this comment

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

ここも同じくです

Suggested change
このようにして、ターミナルで`python`と入力するしたときに、OSは`C:\opt\custompython\bin\python`(リストの末尾のディレクトリ)にあるPythonプログラムを見つけ、使用します。
このようにして、ターミナルで`python`と入力したときに、OSは`C:\opt\custompython\bin\python`(リストの末尾のディレクトリ)にあるPythonプログラムを見つけ、使用します。

@k94-ishi k94-ishi force-pushed the ja-environment-variables.md branch from 7d01a4a to f88ed5e Compare January 26, 2025 05:41
Copy link
Contributor

@k94-ishi
Copy link
Contributor Author

k94-ishi commented Jan 26, 2025

@kiharito レビューいただきありがとうございます!
拙いミスばかり指摘させてしまい申し訳ないです。指摘いただいたどの箇所についても、ご提案の通り修正するのが良いように思いましたので、そのようにしました。
ご確認をおねがいします!🙇

@kiharito
Copy link

@k94-ishi
いえいえ!ありがとうございます!

Copy link

@kiharito kiharito left a comment

Choose a reason for hiding this comment

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

LGTM!

@alejsdev alejsdev merged commit 8f35927 into fastapi:master Jan 27, 2025
50 checks passed
@alejsdev
Copy link
Member

Thanks for your contribution! @k94-ishi 🚀
And thanks for your review @kiharito 🔍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved-1 lang-all Translations lang-ja Japanese translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants