delete unnecessary log - Githubissues

jam2in / arcus-java-client

Arcus Java client

Apache License 2.0

0 stars 0 forks source link

Closed whchoi83 closed 8 years ago

whchoi83 commented 8 years ago

replication 개발 당시 디버그 목적으로 넣었던 log 를 제거하지 않고 release 했습니다. 불필요한 로그라서 삭제합니다.

whchoi83 commented 8 years ago

@aiceru 딱히 리뷰가 필요하진 않지만 본 PR 한 번 봐주시고, 현재 develop 에 있는 commit 들 master 로 merge 되어 release 되어도 문제 없는지 검토 같이 해주세요.

jhpark816 commented 8 years ago

@aiceru 확인하고 merge 하세요..

aiceru commented 8 years ago

다른 commit 들 보니 문제될만한 이슈는 없어 보입니다. develop -> master 로 한번 떨구고 mvn test 진행후 naver/master 로 PR 생성하겠습니다.

jhpark816 commented 8 years ago

jam2in/develop 에서 => naver/develop으로 PR 보내는 것이 어떤가요 ?

jhpark816 commented 8 years ago

향후에 ChangeLog 수정하고 하는 작업을 naver/develop으로 진행한 후에 naver/master로 merge시키려고 합니다..

수행 과정이 어떤가요 ?

whchoi83 commented 8 years ago

naver/develop 을 유지할 필요가 있을까요? 말씀하신 대로 진행하면

jam2in/develop -> jam2in/master
               -> naver/develop -> naver/master

가 되어야 하는데 jam2in/develop -> jam2in/master -> naver/master면 충분할 것 같습니다.

aiceru commented 8 years ago

지금 jam2in/arcus-java-client 에서 release/1.9.5 branch 생성하여 ChangeLog 작업중입니다... 모든 작업은 jam2in repo 에서 수행하고 naver repo 에는 master 만 유지하는 것이 깔끔합니다.