Skip to content

[텍스트 디코더] 번역 본문 번역 완료#338

Closed
kimkyungbae wants to merge 2 commits intojavascript-tutorial:masterfrom
kimkyungbae:kimkyungbae-patch-1
Closed

[텍스트 디코더] 번역 본문 번역 완료#338
kimkyungbae wants to merge 2 commits intojavascript-tutorial:masterfrom
kimkyungbae:kimkyungbae-patch-1

Conversation

@kimkyungbae
Copy link
Contributor

@kimkyungbae kimkyungbae commented Oct 26, 2019

Pull Request 체크리스트

TODO

  • 번역 규칙을 확인하셨나요?
    • 줄 바꿈과 단락을 '원문과 동일하게' 유지하셨나요?
    • 맞춤법 검사기로 맞춤법을 확인하셨나요?
    • 공백(스페이스), 큰따옴표("), 작은따옴표('), 대시(-), 백틱(`)을 비롯한 모든 특수문자는 그대로 두셨나요?
  • 로컬 서버 세팅 후 최종 결과물을 확인해 보셨나요?
  • PR 하나엔 번역문 하나만 넣으셨나요?
  • 의미 있는 커밋 메시지를 작성하셨나요?
    • 예시
      • [프락시] 번역
      • [프락시] 과제 번역
      • [if문과 조건부 연산자 '?'] 리뷰
      • [주석] 2차 리뷰
      • [Date 객체와 날짜] 번역

@CLAassistant
Copy link

CLAassistant commented Oct 26, 2019

CLA assistant check
All committers have signed the CLA.

@JuYeong0413
Copy link
Contributor

JuYeong0413 commented Oct 26, 2019

Pull Request 체크리스트의 의미있는 커밋 메세지 작성 부분을 체크하셨는데요, 커밋 메세지를 보면 파일 경로를 적어두신 것으로 보입니다. 파일 경로는 커밋 메시지가 아니더라도 알 수 있기 때문에 커밋 메세지에는 파일 경로 이름을 잘 쓰지 않습니다. 예시처럼 [텍스트 디코더] 번역 으로 변경하시면 어떨까요?

@kimkyungbae kimkyungbae changed the title 4-binary/02-text-decoder/article.md 본문 번역 완료 [텍스트 디코더] 번역 본문 번역 완료 Oct 26, 2019
Update 4-binary/02-text-decoder/article.md

Co-Authored-By: Ju Yeong <33684401+JuYeong0413@users.noreply.github.com>

Update 4-binary/02-text-decoder/article.md

Co-Authored-By: Ju Yeong <2jy22@naver.com>

Update 4-binary/02-text-decoder/article.md

Co-Authored-By: Ju Yeong <2jy22@naver.com>

Update 4-binary/02-text-decoder/article.md

Co-Authored-By: Ju Yeong <2jy22@naver.com>

Update 4-binary/02-text-decoder/article.md

Co-Authored-By: Ju Yeong <2jy22@naver.com>

Update 4-binary/02-text-decoder/article.md

Co-Authored-By: Ju Yeong <2jy22@naver.com>

Update 4-binary/02-text-decoder/article.md

Co-Authored-By: Ju Yeong <2jy22@naver.com>
@JuYeong0413
Copy link
Contributor

JuYeong0413 commented Oct 26, 2019

수정하신 것은 커밋 메세지가 아니라 Pull Request 제목입니다.

@kimkyungbae
Copy link
Contributor Author

수정하신 것은 커밋 메세지가 아니라 Pull Request 제목입니다.

커밋 -메세지 바꿨습니다 !

모든 출력값은 입력값을 넣는 즉시 변경되어야 합니다.

The formula is:
공식:
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
공식:
공식은 아래와 같습니다.

@@ -1,18 +1,18 @@
importance: 5
중요도: 5
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

@JuYeong0413 JuYeong0413 left a comment

Choose a reason for hiding this comment

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

글 번역과 과제 번역 PR을 분리하는걸 권유드립니다.
README

The formula is:
공식:
```js
// initial: the initial money sum
Copy link
Contributor

Choose a reason for hiding this comment

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

주석 부분 번역 누락되었습니다.

@kimkyungbae kimkyungbae deleted the kimkyungbae-patch-1 branch October 27, 2019 06:26
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.

4 participants

Comments