githru / githru-vscode-ext

Lightweight but robust Githru for VSCode Extension
Apache License 2.0
49 stars 83 forks source link

[all] lint/prettier husky, lint-staged 적용 #722

Closed choisohyun closed 3 weeks ago

choisohyun commented 1 month ago

Related issue

close #662

Result

Screenshot 2024-09-11 at 10 03 02 PM Screenshot 2024-09-11 at 10 58 47 PM

Screenshot 2024-09-11 at 11 03 16 PM

Work list

Discussion

xxxjinn commented 1 month ago

음 허스키 적용은 전체 협의(엔진, vscode, view 모두) 후에 결정하면 좋을 것 같습니다. 이게 호불호가 있는 부분이기도 하고, package 별로 정책이 조금씩 다르기도 하구요.

아니면 우선 engine 부분만 적용해보는건 어떨까요?

저두 이 말씀에 동의합니다.! 개인적으로 husky를 쓸 때 자꾸 정해둔 걸 까먹거나 해서 번거로운 경우가 있어서 선호하지는 않는 편이지만, 한 편으로는 오픈소스 기여라면 아무래도 여러 사람들이 커밋을 올리니 그 형식이 통일되는 것이 깔끔하고 좋을 것 같아요🙂

ytaek commented 1 month ago

@choisohyun 님, 커멘트 적힌 대로 husky 적용 부분은 engine 팀과 얘기해보시고 반영 여부를 결정해주시면 감사하겠습니다!

choisohyun commented 3 weeks ago

우선 close 하겠습니다!