TIL/WID: What I Did

2월 26일 WID : PR 메시지 포맷, button 규칙 지키기

반응형

PR 메시지 포맷

PR 메시지 포맷에 대해 생각했다. 커밋과 PR의 단위에 대해서도 좀 생각해볼만 할 것 같다. 요구사항이 큰 덩어리인 경우에는 PR과 커밋메시지가 확실하게 다른 계층으로 분리될 수 있을 거 같은데 작은 덩어리인 경우엔 PR과 커밋 메시지의 레벨이 동일할 수도 있겠다는 생각이 들었다. 이런 경우라면 차이점은 PR에는 상세한 내역을 기록하고, 커밋 메시지는 일종의 요약본으로 생각할 수도 있지 않을까? 커밋 메시지에 딸린 PR내역을 추적하긴 쉬우니까.. 
오늘은 아래와 같은 포맷으로 작성했다. 

PR 제목

문제상황

  • 어떠어떠한 상태인 경우, 이러저러한 문제가 발생한다.

해결방법

  • 어떤 로직을 추가하여 해결하였다.

참고사항

  • 같은 문제가 이전에 OO에서도 발생한 바 있으며, 당시 원인은 정확히 파악하지 못했다.

button 규칙 지키기

오늘 소스코드를 돌아보다가 div를 사용하여 버튼처럼 만들어놓은 컴포넌트를 발견했다. onClick은 button이나 링크에만 들어가는게 기본이고, 굳이굳이 button이 아닌 엘리먼트에 onClick을 적용해 버튼처럼 쓰고 싶다면 role="button"이라도 부여해줘야 한다는 말이 기억났다. 그래서 button으로 바꿔주고 커밋했다. 

728x90
반응형