Skip to content

2018-edition의 ch18-02 오타 수정 #237

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

Merged
merged 4 commits into from
Dec 30, 2020
Merged

Conversation

soilSpoon
Copy link

@rinthel
Copy link
Owner

rinthel commented Dec 30, 2020

@soilSpoon 번역본 문서가 원본 문서와 완전 다르게 될수 밖에 없는 이상 업데이트 기준이 계속 바뀌면 문제가 생길수 있습니다- 가급적이면 처음에 업데이트했던 #77092be 커밋을 기준으로 부탁드릴게요. 지금의 #3b3afd3 커밋은 오타 수정 관련 커밋으로 이해하겠습니다-

아울러서, 같은 성질의 풀리퀘스트는 가급적 하나로 묶어서 업데이트 요청 드리겠습니다~

@soilSpoon
Copy link
Author

soilSpoon commented Dec 30, 2020

@rinthel 이 PR은 77092be 이전에 올라온 오타 수정인데 당시 모든 파일들이 파악이 안 되어서 확실하게 올려도 될 것 같은 파일만 올렸던거라 이렇게 나중에 올리게 되었습니다.

보통 다른 레포들은 한 작업 단위로 올려서 작업 단위로 올렸었는데 지금 올라가있는 #238, #239 도 이 PR에 합쳐서 올리는게 나을까요?

@rinthel
Copy link
Owner

rinthel commented Dec 30, 2020

@soilSpoon 넵 제가 보기에는 결국 번역된 마크다운을 제외한 모든 문서에 대한 원본과의 동기화라는 점에서 동일한 작업인것 같아서요 ㅎㅎ 변경된 파일이 많긴 하지만 구분이 크게 필요는 없을듯 합니다

@soilSpoon soilSpoon marked this pull request as draft December 30, 2020 03:24
@soilSpoon
Copy link
Author

soilSpoon commented Dec 30, 2020

@rinthel 동기화하기 애매한 작업이 남았는데 2nd본에서 빌드 관련 파일들을 원본 레포에서는 1st본처럼 폐지?되었는데 이러면 안 될 것 같아서요.

지금 이대로 유지하는게 좋을까요? 아니면 원본 레포의 최상위 빌드 환경과 같이 맞추는 게 좋을까요?

@rinthel
Copy link
Owner

rinthel commented Dec 30, 2020

@soilSpoon 음 일단은 그대로 둬야 할것 같습니다. 2018년 버전 작업이 좀 많이 남았으니 2018년 버전을 빌드한 문서를 바로 공개할 수는 없고, 일부 번역만이라도 공개하려고 해도 2판과 2018판 모두 공개해두는게 맞을것 같습니다. 만약에 업데이트를 한다면 현재의 second-edition 디렉토리를 다른쪽으로 옮기고 개별적으로 빌드를 할 수 있도록 하거나 해야할것 같구요. second-edition 디렉토리의 동기화가 아직 남았음을 별도의 이슈로 뽑겠습니다.

@soilSpoon soilSpoon marked this pull request as ready for review December 30, 2020 04:59
@soilSpoon
Copy link
Author

soilSpoon commented Dec 30, 2020

@rinthel merge 준비 완료되었습니다.


추가적인 이슈로는 src/ch03-02-data-types - 번역.md라는 파일이 존재하는데 번역하는 중에 멈춘 것 같은데 어떡해야 될지 몰라 남겨두었습니다.

@rinthel
Copy link
Owner

rinthel commented Dec 30, 2020

@soilSpoon 넵 번역중인 문서는 추후 번역이 된 후에 제거하는 것으로 하겠습니다

@rinthel rinthel merged commit a390f61 into rinthel:master Dec 30, 2020
@soilSpoon soilSpoon deleted the fix-typo branch December 30, 2020 05:51
@rinthel rinthel mentioned this pull request Dec 30, 2020
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants