-
Notifications
You must be signed in to change notification settings - Fork 68
update a policy for nodejs-ko maintainers #479
Conversation
|
||
# [nodejs-ko](https://github.com/orgs/nodejs/teams/nodejs-ko) 메인테이너 | ||
* 메인테이너는 [nodejs-ko 팀](https://github.com/orgs/nodejs/teams/nodejs-ko)에 등록된 멤버를 의미합니다. | ||
* 번역을 참여하기 위해 Collaborators로 등록되더라도 nodejs-ko 팀에 등록되지는 않습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Collaborators와 메인테이너를 나눠서 설명하는건 어떨까요?
대략..
# 맴버 구분
## nodejs/nodejs-ko Collaborators
* 레포지터리에 소속된 맴버이며 nodejs 조직이나 nodejs-ko 팀에는 등록 되지않습니다.
* nodejs-ko에 올라오는 번역문의 리뷰, 머지에 대한 권한을 가집니다.
* ....
## [nodejs-ko](https://github.com/orgs/nodejs/teams/nodejs-ko) 메인테이너
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저도 이 의견에 동의합니다.
* nodejs-ko 메인테이너는 nodejs-ko 번역문의 리뷰(Collaborators도 리뷰가 가능합니다.)외에 저장소 관리 및 Node.js 아래 다른 프로젝트에서 한국어 관련 이슈에 대한 논의에 참여합니다. | ||
* 다른 프로젝트에서 nodejs-ko 의 의견을 듣기 위해서 멘션을 하는 경우가 있습니다. | ||
* nodejs-ko 저장소 외에 [nodejs.org](https://github.com/nodejs/nodejs.org)의 한국어 번역과 검수에도 참여하고 있습니다. | ||
* nodejs-ko 저장소 관리를 위해 번역문이 아닌 간단한 맞춤법 오류, 서식 오류, 오타, 저장소 설정 등은 풀 리퀘스트를 거치지 않고 바로 수정할 수 있습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
처음 논의했던 대로 간단한 오류 등의 경우 빠른 반영을 위해 다른 분의 풀 리퀘스트를 수정할 수도 있다고 명시하였으면 합니다.
|
||
# [nodejs-ko](https://github.com/orgs/nodejs/teams/nodejs-ko) 메인테이너 | ||
* 메인테이너는 [nodejs-ko 팀](https://github.com/orgs/nodejs/teams/nodejs-ko)에 등록된 멤버를 의미합니다. | ||
* 번역을 참여하기 위해 Collaborators로 등록되더라도 nodejs-ko 팀에 등록되지는 않습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
번역을 -> 번역에
# [nodejs-ko](https://github.com/orgs/nodejs/teams/nodejs-ko) 메인테이너 | ||
* 메인테이너는 [nodejs-ko 팀](https://github.com/orgs/nodejs/teams/nodejs-ko)에 등록된 멤버를 의미합니다. | ||
* 번역을 참여하기 위해 Collaborators로 등록되더라도 nodejs-ko 팀에 등록되지는 않습니다. | ||
* nodejs-ko 메인테이너는 nodejs-ko 번역문의 리뷰(Collaborators도 리뷰가 가능합니다.)외에 저장소 관리 및 Node.js 아래 다른 프로젝트에서 한국어 관련 이슈에 대한 논의에 참여합니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
나름대로 조금 수정해보았습니다.
메인테이너는 nodejs-ko 문서 번역, 번역문의 리뷰(Collaborators도 리뷰는 가능), 저장소 관리는 물론 Node.js 소속 다른 프로젝트에서 발생 또는 요청하는 한국어 관련 이슈에도 참여합니다.
* nodejs-ko 저장소 외에 [nodejs.org](https://github.com/nodejs/nodejs.org)의 한국어 번역과 검수에도 참여하고 있습니다. | ||
* nodejs-ko 저장소 관리를 위해 번역문이 아닌 간단한 맞춤법 오류, 서식 오류, 오타, 저장소 설정 등은 풀 리퀘스트를 거치지 않고 바로 수정할 수 있습니다. | ||
* nodejs-ko의 메인테이너는 6개월마다 갱신하며 갱신 시점을 기준으로 최근 1년간 nodejs-ko 저장소에 기여를 한 사람을 메인테이너로 등록합니다. | ||
* 메인테이너는 Node.js 릴리스 주기에 맞춰서 매년 4월, 10월에 갱신합니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
메인테이너는 -> 메인테이너 명단은
* 다른 프로젝트에서 nodejs-ko 의 의견을 듣기 위해서 멘션을 하는 경우가 있습니다. | ||
* nodejs-ko 저장소 외에 [nodejs.org](https://github.com/nodejs/nodejs.org)의 한국어 번역과 검수에도 참여하고 있습니다. | ||
* nodejs-ko 저장소 관리를 위해 번역문이 아닌 간단한 맞춤법 오류, 서식 오류, 오타, 저장소 설정 등은 풀 리퀘스트를 거치지 않고 바로 수정할 수 있습니다. | ||
* nodejs-ko의 메인테이너는 6개월마다 갱신하며 갱신 시점을 기준으로 최근 1년간 nodejs-ko 저장소에 기여를 한 사람을 메인테이너로 등록합니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- 기여를 한 사람 -> 기여한 사람
- 읽기에 따라 기여한 사람 전부가 메인테이너가 된다라고 보일 것 같은데, 그렇게 의도하신 건가요?
56fad9d
to
d710b03
Compare
전부라고 하면 6개월에 한 번 글을 번역하신 분도 공식 멤버가 되는 건데 그보다는 활동이 더 있어야 하지 않을까 싶습니다. 혹시 번역뿐 아니라 리뷰도 카운팅할 수 있을까요? 방법이 있다면 좋을텐데요. ^^;; |
리뷰는 확인할 방법이 따로 있는지 모르겠네요. 커밋 기준으로 보면 https://github.com/nodejs/nodejs-ko/graphs/contributors?from=2016-01-01&to=2016-12-30 1년 기준으로 했을때 10회 정도면 어떨까 싶기도 합니다. 한 PR에 커밋 많은 분도 있고 아닌 사람도 있지만요. |
|
||
# 멤버 구분 | ||
|
||
## [nodejs-ko](https://github.com/nodejs/nodejs-ko) Collaborators |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
팀과 저장소의 구별이 안되네요. 제목에 링크를 빼고 밑에 줄의 [nodejs-ko]
저장소를 [nodejs-ko 저장소]
로 바꾸는건 어떨까요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네 동의합니다.
파라메터 수정하면 6개월간도 볼수 있습니다. https://github.com/nodejs/nodejs-ko/graphs/contributors?from=2016-06-01&to=2016-12-31&type=c |
번역글이 아주 많지는 않으니 저는 6개월마다 목록을 업데이트하지만 기준 기간을 1년을 생각하고 있었습니다. 6개월의 기간은 계속 겹치는 시간으로요. 6개월 기준으로 하면 기준을 낮춰도 되긴 합니다. 1년 기준으로 해서 커밋 10개로 기준으로 하면 어떨까요? 그냥해도 큰 문제는 없을 것 같고 글 1개인데 커밋이 너무 많은 경우에는 squash 머지를 해도 될것 같고요. (아직 목록 업데이트를 할 수 없는 상황이긴 하지만요 ㅎㅎ ) |
d710b03
to
30dca69
Compare
얼마 전부터 제가 머지할 때는 로그가 깔끔하다는 이유로 Squash 머지하고 있었는데 그렇게 되면 커밋 수=번역한 글 수가 될 수도 있겠네요. 지난 1년 기준 10개 정도면 적당한 기준이라 생각합니다. 다만, 리뷰도 잘 이루어졌으면 하는 마음에 가능하다면 리뷰도 기준에 포함시켰으면 하는 생각이 있습니다. ^^; |
저도 리뷰를 포함했으면 하는데 뭔가 방법이 깔끔하지 않네요. |
30dca69
to
31b9804
Compare
* 빠른 반영을 위해 간단한 오류 등은 다른 사람이 등록한 풀 리퀘스트를 직접 수정할 수 있습니다. | ||
* nodejs-ko의 메인테이너는 6개월마다 갱신하며 갱신 시점을 기준으로 최근 1년간 nodejs-ko 저장소에 기여한 사람을 메인테이너로 등록합니다. | ||
* 메인테이너 명단은 Node.js 릴리스 주기에 맞춰서 매년 4월, 10월에 갱신합니다. | ||
* [Graph](https://github.com/nodejs/nodejs-ko/graphs/contributors)를 기준으로 최근 1년간 커밋이 10개 이상인 사람을 메인테이너도 등록합니다.(풀 리퀘스트는 Squash로 머지합니다.) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
논의가 어느정도 된것 같아서 10개 커밋 기준이라는 내용을 추가했습니다.
* [nodejs-ko 저장소](https://github.com/nodejs/nodejs-ko)에 Collaboratios로 등록된 사용자를 말하며 nodejs 조직이나 [nodejs-ko 팀](https://github.com/orgs/nodejs/teams/nodejs-ko)에는 등록되지 않습니다. | ||
* [nodejs-ko 저장소](https://github.com/nodejs/nodejs-ko)에 저장소에 올라오는 이슈를 관리할 수 있고 번역문의 리뷰, 머지 권한을 가집니다. | ||
|
||
# nodejs-ko Maintainers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
혹시 node.js-ko Maintainers가 멤버 구분
의 하위제목인가요?
위에 있는 nodejs-ko Collaborators는 ##으로 시작하는데, 해당 부분은 # 한개인 것같아 코멘트 남겨봅니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아 그러네요.
* 빠른 반영을 위해 간단한 오류 등은 다른 사람이 등록한 풀 리퀘스트를 직접 수정할 수 있습니다. | ||
* nodejs-ko의 메인테이너는 6개월마다 갱신하며 갱신 시점을 기준으로 최근 1년간 nodejs-ko 저장소에 기여한 사람을 메인테이너로 등록합니다. | ||
* 메인테이너 명단은 Node.js 릴리스 주기에 맞춰서 매년 4월, 10월에 갱신합니다. | ||
* [Graph](https://github.com/nodejs/nodejs-ko/graphs/contributors)를 기준으로 최근 1년간 커밋이 10개 이상인 사람을 메인테이너도 등록합니다.(풀 리퀘스트는 Squash로 머지합니다.) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
메인테이너도 → 메인테이너로 인 것 같습니다.
31b9804
to
4f49919
Compare
수정했습니다. |
슬슬 머지하죠 'ㅁ' |
더이상 이견이 없는것 같고 이후에도 수정가능하니 머지하겠습니다. |
close #456
#456 에서 논의한 내용을 기여가이드 문서에 추가했습니다.
해당 내용을 넣기 위해서 #425 에서 논의한 메인테이너 관리에 대한 정책도 추가했습니다.
일단 제가 임의로 적은 내용이니 수정할 내용이나 보충할 내용있으면 알려주세요.