fora22 commented 3 years ago

1. Issue 확인 문제

PR은 review 라는 형태가 있지만 Issue는 상대가 확인했는지 알기 힘듭니다. 그래서 명시적으로 규칙이 있으면 좋을 것 같아서 제안해봅니다.

'확인했습니다'라고 Comment를 일일히 남기는 것 보다는 이모티콘을 통해 확인의사를 표현하는 게 나을 것 같습니다.
여기서 확인은 'Issue 가 올라왔구나'가 아닌 'Issue를 읽고 내용에 대해 인지를 했을 때'를 말합니다.
- '올라온 것만 알고있다'는 안됨!!

다만 이 적용 대상은 Assignees or Reviewers 에 해당합니다.

2. 긴급한 이슈 or Review 에 대한 Labeling

긴급하게 확인 해줬으면 하는 이슈는 '[긴급]'이라고 라벨링 하는게 어떨까요? 암묵적(?)으로 가끔 하고 있지만 규칙으로 명시해두는게 나아 보여서요.

물론 시간제한 같은 강제성은 없지만 이슈의 성격을 구분하기 위한 조치입니다.

[긴급] 이슈에 대해 다른 의견이 있으면 의견 남겨주시면 감사하겠습니다.

dkfma4915 commented 3 years ago

두가지 의견 모두 좋습니다~~ Issue 제목에 [긴급]이라고 맨 앞에 쓰자는거지?! Label에도 긴급과 관련된 Label 추가해서 제목, Label둘다 해도 좋을듯!

fora22 commented 3 years ago

Issue 제목에 [긴급]이라고 맨 앞에 쓰자는거지?! Label에도 긴급과 관련된 Label 추가해서 제목, Label둘다 해도 좋을듯!

오 좋아!

지금 Label 들이 다 영어로 써져 있으니 승인되면 Emergency로 `빨간색 해서 추가하는 쪽으로 진행해볼게!

fora22 commented 3 years ago

그런데 이 내용과 상관 없는 sql 스크립트가 추가가 됐네요. 이번 pull request와 상관 없는 파일은 별도로 추가하면 좋을 것 같습니다.

작업 중에 커밋하다 보니 추가가 됐네요. 반영 하겠습니다!

fora22 commented 3 years ago

@jongfeel 상관없는 파일 별도로 구분하였습니다. Review Request 합니다!