Skip to content

Comments

intro 2차 리뷰#244

Merged
Violet-Bora-Lee merged 2 commits intojavascript-tutorial:masterfrom
sojeongw:master
Oct 2, 2019
Merged

intro 2차 리뷰#244
Violet-Bora-Lee merged 2 commits intojavascript-tutorial:masterfrom
sojeongw:master

Conversation

@sojeongw
Copy link
Contributor

@sojeongw sojeongw commented Oct 1, 2019

정리해서 다시 PR 날립니다.

@Violet-Bora-Lee
Copy link
Member

upstream(저희 저장소)을 반영해주셔야해요. 지금은 충돌상태라 머지가 안됩니다.

@Violet-Bora-Lee
Copy link
Member

@sojeongw
Copy link
Contributor Author

sojeongw commented Oct 2, 2019

죄송합니다. 회사 일로 수정이 늦었습니다. upstream 반영했습니다.

@Violet-Bora-Lee
Copy link
Member

회사일로 바쁘신 가운데 기여해주셔서 감사합니다.
👍 👍 👍

@Violet-Bora-Lee
Copy link
Member

Violet-Bora-Lee commented Oct 2, 2019

다음엔 upstream을 받아올 때, pull(==fetch+merge)말고, fetch후 rebase하는 방식으로 도전해보세요~
이번 PR은 제가 squash해서 반영하겠습니다 :)

@Violet-Bora-Lee Violet-Bora-Lee merged commit bb0f32f into javascript-tutorial:master Oct 2, 2019
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.

3 participants