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

1-js/03-code-quality/05-testing-mocha- review #273

Closed
wants to merge 6 commits into from
Closed

1-js/03-code-quality/05-testing-mocha- review #273

wants to merge 6 commits into from

Conversation

patiencendiligence
Copy link

피동형 수정, 호응 구조 수정, 어감 수정, markdown 수정

…sted

피동형 수정
호응 구조 수정
어감 수정
1-js/01-An Introduction/2-Manuals and specifications 리뷰  changes requ…
피동형 수정, 호응 구조 수정, 어감 수정, markdown 수정
1-js/03-code-quality/05-testing-mocha- review
@CLAassistant
Copy link

CLAassistant commented Oct 11, 2019

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.


그런데 이렇게 수동으로 코드를 "재실행"하는 건 상당히 불완전합니다.
하지만 이러한 수동 "재실행" 불완전합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

'재실행'으로 바꾸는 건 어떨까요? #245

@JuYeong0413
Copy link
Contributor

매뉴얼과 명세서 부분 커밋이 함께 올라와 있는데, PR을 따로 작성해주시기 바랍니다. 😃
PR 하나에 한 챕터 작업을 권유드리고 있습니다. README

@Violet-Bora-Lee
Copy link
Member

브랜치를 나눠서 작업하는 연습을 하셔야 할 것 같습니다.
@JuYeong0413 님께서 말씀해주신것처럼 본 PR에는 관련 커밋만 있도록 해주세요.

@patiencendiligence
Copy link
Author

아하 네네 한번에 커밋하는 습관이 있어서 저도모르게 일괄로 올려버렸네요 ㅠㅠ

@Violet-Bora-Lee
Copy link
Member

Violet-Bora-Lee commented Oct 28, 2019

주제별로 브랜치 분리하여 작업 부탁드립니다.
본 PR은 크로즈하겠습니다.

소스 최신으로 업데이트하셔야할거예요 #333

위 리뷰 먼저 반영하려 합니다.

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.

5 participants