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

🌐 Add Korean translation for docs/ko/docs/contributing.md #4019

Closed
wants to merge 6 commits into from
Closed

🌐 Add Korean translation for docs/ko/docs/contributing.md #4019

wants to merge 6 commits into from

Conversation

kimjaeyoonn
Copy link
Contributor

This PR translates contribuing.md in Korean.

I understand it as a request to divide a branch, and divide the Branch and post a PR again.
Thank You, I look forward to your feedback.

related: #2017

Copy link
Contributor

@0417taehyun 0417taehyun left a comment

Choose a reason for hiding this comment

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

I think writing english words less would be better, like python, markdown and some headers and tip part are not translated yet. So please check my reviews.

Little bit hard to check the all of the sentences, words, etc. carefully. So, I'm in progress to check and share my opinion, and also those are not the answers. Just free to check mine 🚀


## 개발

이미 저장소를 Clone했고, 코드를 조사하게 알고싶은 경우에 환경 구축을 위한 몇 가지 가이드라인이 있습니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

How about 복제 instead of Clone?


### `venv`를 이용한 가상환경

Python의 `venv` 모듈을 사용하여 디렉토리에 가상 환경을 만들 수 있습니다:
Copy link
Contributor

Choose a reason for hiding this comment

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

How about 파이썬 instead of Python?

If you check 자습서 - 사용자 안내서 - 도입부, the word Python is translated as 파이썬.

$ python -m venv env
```

그러면 Python 바이너리를 포함한 `./env/` 디렉토리가 생성되고 그 고립된 환경에 패키지를 설치할 수 있습니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here, 파이썬 instead of Python.

</div>
````

동작 확인을 위해 다음을 실행하십시오:
Copy link
Contributor

Choose a reason for hiding this comment

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

How about 작동하는 걸 확인하기 위해 instead 동작 확인을 위해?

동작 means for living things movement such as human and 작동 means for machines.

Check 국립국어원: <동작과 작동 구분> 질문에 대한 답변


`env/bin/pip` 에 `pip` 바이너리가 나타난다면 제대로 작동하고 있는 것입니다. 🎉

!!! tip 이 환경 아래에 `pip` 에서 새 패키지를 설치할 때마다, 가상 환경을 다시 활성화 합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it would be better to add "팁" right side of tip.

Check #3167 section 파이썬 공식 한국어 문서에 없는 용어 중 현재 사용하고 있는 용어.


이제 모든 것을 번역할 수 있고, 파일의 저장 상태를 확인할 수 있습니다.

#### New Language
Copy link
Contributor

Choose a reason for hiding this comment

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

The header was not translated yet.


그 후 "기존의 언어"에 대한 이전의 지시를 계속할 수 있습니다.

##### New Language not supported
Copy link
Contributor

Choose a reason for hiding this comment

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

The header was not translated yet.


그 후 라이브 서버를 다시 시작합니다.

#### Preview the result
Copy link
Contributor

Choose a reason for hiding this comment

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

The header was not translated yet.

Serving at: http://127.0.0.1:8008
```

## Tests
Copy link
Contributor

Choose a reason for hiding this comment

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

The header was not translated yet.


### Flit

**FastAPI** 는 [Flit](https://flit.readthedocs.io/en/latest/index.html) 을 사용하여 프로젝트를 빌드, 패키지, publish합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

How about 공개 instead of publish?

Because in the context, this (FastAPI) is the open source project so it is open to public by publish the project.

@tiangolo tiangolo changed the title 🌐 Add Korean translation for docs/contributing.md 🌐 Add Korean translation for docs/ko/docs/contributing.md Oct 26, 2021
@kimjaeyoonn
Copy link
Contributor Author

@tiangolo

This PR document has been modified and uploaded again in #4143, so please check it and merge it.

@DevDae
Copy link
Contributor

DevDae commented Nov 19, 2021

@tiangolo

This PR document has been modified and uploaded again in #4143, so please check it and merge it.

I just have reviewed this PR now.
It doesn't have any flaw! 🚀 Please check our reviews 🙏

joonas-yoon added a commit to joonas-yoon/fastapi that referenced this pull request Aug 6, 2022
@tiangolo
Copy link
Owner

As this PR had requested changes to be applied but has been inactive for a while, it's now going to be closed. But if there's anyone interested, feel free to create a new PR.

@tiangolo tiangolo closed this Jan 16, 2024
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.

None yet

5 participants