Skip to content

#1717 random-min-max solution 번역 5번째 줄 #1729

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.

Sign up for GitHub

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

Jump to bottom
Merged
Violet-Bora-Lee merged 1 commit into from
Dec 10, 2023
Merged

#1717 random-min-max solution 번역 5번째 줄 #1729

Violet-Bora-Lee merged 1 commit into from
Dec 10, 2023

Conversation

@cocoa-salon
Copy link
Contributor

@cocoa-salon cocoa-salon commented Dec 1, 2023 •
edited
Loading

요약

random-min-max solution 번역 5번째 줄

연관 이슈

#1717

Pull Request 체크리스트

TODO

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

@javascript-translate-bot javascript-translate-bot requested a review from a team December 1, 2023 07:21
@CLAassistant
Copy link

CLAassistant commented Dec 1, 2023 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

Violet-Bora-Lee
Violet-Bora-Lee approved these changes Dec 1, 2023
View reviewed changes
Copy link
Member

@Violet-Bora-Lee Violet-Bora-Lee left a comment

Choose a reason for hiding this comment

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

잘 해주셨습니다.
다만 조금더 독자를 위해 개선해주셨으면 하는 부분에 대해 코멘트 달아보았습니다.

@cocoa-salon
Copy link
Contributor Author

cocoa-salon commented Dec 1, 2023 •
edited
Loading

피드백 반영하여 origin/1717-random-min-max-solution-5 에 푸시하였습니다. 확인 부탁드립니다.

@Violet-Bora-Lee
Copy link
Member

Violet-Bora-Lee commented Dec 6, 2023 •
edited
Loading

커밋을 하나로 합쳐주세요.
작업 규모에 비해 커밋 갯수가 너무 많습니다.
merge 커밋도 없애주세요.

Violet-Bora-Lee
Violet-Bora-Lee requested changes Dec 6, 2023
View reviewed changes
Copy link
Member

@Violet-Bora-Lee Violet-Bora-Lee left a comment

Choose a reason for hiding this comment

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

코멘트 달아두었습니다.

@javascript-translate-bot
Copy link

javascript-translate-bot commented Dec 6, 2023

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@cocoa-salon cocoa-salon force-pushed the 1717-random-min-max-solution-5 branch 2 times, most recently from d057f9d to 3fb8878 Compare December 6, 2023 09:29
@cocoa-salon
Copy link
Contributor Author

cocoa-salon commented Dec 6, 2023

@Violet-Bora-Lee 네 커밋을 하나로 합쳐서 다시 push 하였습니다. 확인 부탁드립니다.

@cocoa-salon
Copy link
Contributor Author

cocoa-salon commented Dec 6, 2023

@Violet-Bora-Lee 파일을 저장할 때 자동 포매팅 기능이 실행되면서 다른 부분의 코드의 포맷이 변경되었는데요. 다시 수정하겠습니다.

@cocoa-salon cocoa-salon force-pushed the 1717-random-min-max-solution-5 branch from 2afdfbc to a71ad14 Compare December 6, 2023 09:51
@Violet-Bora-Lee
Copy link
Member

Violet-Bora-Lee commented Dec 6, 2023 •
edited
Loading

커밋과 이슈를 연동해주세요. @cocoa-salon

커밋메시지에 #이 빠져있습니다.

@cocoa-salon cocoa-salon force-pushed the 1717-random-min-max-solution-5 branch from a71ad14 to dc6b509 Compare December 6, 2023 09:56
@cocoa-salon
Copy link
Contributor Author

cocoa-salon commented Dec 6, 2023

커밋과 이슈를 연동해주세요. @cocoa-salon

커밋메시지에 #이 빠져있습니다.

네 반영하여 푸시하였습니다.

Violet-Bora-Lee
Violet-Bora-Lee requested changes Dec 6, 2023
View reviewed changes
1-js/05-data-types/02-number/8-random-min-max/solution.md Outdated
Comment on lines 19 to 18

``` No newline at end of file
Copy link
Member

@Violet-Bora-Lee Violet-Bora-Lee Dec 6, 2023

Choose a reason for hiding this comment

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

여기는 왜 변경된거죠?

PR 리뷰 요청을 하기전에 자신이 만든 PR을 점검해주세요.

Copy link
Contributor Author

@cocoa-salon cocoa-salon Dec 6, 2023

Choose a reason for hiding this comment

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

자동 포맷팅 기능으로 인한 기존 코드 포맷 변경 사항 복원이 완전히 이루어지지 않은 것 같습니다. 다시 수정하겠습니다.

@javascript-translate-bot
Copy link

javascript-translate-bot commented Dec 6, 2023

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@cocoa-salon cocoa-salon force-pushed the 1717-random-min-max-solution-5 branch from dc6b509 to 5abec57 Compare December 6, 2023 10:12
@Violet-Bora-Lee
Copy link
Member

Violet-Bora-Lee commented Dec 10, 2023

그냥 머지해도 되긴하는데,,
위에 코멘트 달린거 확인하셨나요?
PR을 만들게되면 항상 메일링을 통해 PR을 추적하셔야합니다.

cocoa-salon
cocoa-salon commented Dec 10, 2023
View reviewed changes
Copy link
Contributor Author

@cocoa-salon cocoa-salon left a comment

Choose a reason for hiding this comment

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

기존 자동 포맷이 적용된 코드를 복원 후 실제 번역 수정된 부분만 반영하였습니다.

@Violet-Bora-Lee
Copy link
Member

Violet-Bora-Lee commented Dec 10, 2023

Please make the requested changes. After it, add a comment "/done". Then I'll ask for a new review 👻

@cocoa-salon 여기요..

@Violet-Bora-Lee
Copy link
Member

Violet-Bora-Lee commented Dec 10, 2023

PR을 만들었을땐 항상 댓글을 확인해주세요. 요즘엔 bot이 자동으로 다는경우도 많아서 오픈소스 기여할땐 항상 확인하셔야합니다.

@cocoa-salon
Copy link
Contributor Author

cocoa-salon commented Dec 10, 2023

/done

@javascript-translate-bot javascript-translate-bot requested a review from a team December 10, 2023 14:22
@Violet-Bora-Lee
Copy link
Member

Violet-Bora-Lee commented Dec 10, 2023

머지 진행하겠습니다~
한줄짜리 기여인데도 챙길게 많았네요...
앞으론 항상 꼼꼼하게 확인하셔서 오픈소스 잘 기여하시길 바랍니다~

@Violet-Bora-Lee Violet-Bora-Lee merged commit 6f0bc0d into javascript-tutorial:master Dec 10, 2023
@cocoa-salon
Copy link
Contributor Author

cocoa-salon commented Dec 10, 2023

@Violet-Bora-Lee 네 끝까지 지도해주셔서 감사합니다. 늦은 시간까지 고생 많으셨습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Violet-Bora-Lee Violet-Bora-Lee Violet-Bora-Lee requested changes

Assignees

No one assigned

Labels

needs +1 review needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@cocoa-salon @CLAassistant @Violet-Bora-Lee @javascript-translate-bot