Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: eslint, prettier, husky, ci 추가 #3

Merged
merged 4 commits into from
Mar 10, 2023

Conversation

bytrustu
Copy link
Member

@bytrustu bytrustu commented Mar 9, 2023

작업 주제

optional 작업 내용(어떤 부분을 리뷰어가 집중해서 봐야할까요?)

  • airbnb 컨벤션에 따라 eslint와 prettier를 추가했습니다. 62fce6d
  • husky pre-commit을 사용하여 eslint와 prettier 룰을 체크했습니다. 1b67199
  • ci를 사용하여 eslint와 prettier 룰을 체크했습니다. b484b9e

체크리스트(PR 올리기 전 아래의 내용을 확인해주세요.)

  • base, compare branch가 적절하게 선택되었나요?
  • 로컬 환경에서 충분히 테스트 하셨나요?

리뷰 규칙

  • P1: 꼭/적극적 반영해 주세요 (Request changes)
  • P2: 웬만하면 반영해 주세요 (Comment)
  • P3: 반영해도 좋고 넘어가도 좋습니다 (Approve)

@bytrustu bytrustu merged commit bb1ded4 into main Mar 10, 2023
@leeyun1533
Copy link
Collaborator

좋습니다! husky 같은 경우엔 대형 프로젝트엔 유효할 수 있지만, 소규모 프로젝트엔 병목이 될 수 있다는 생각도 있습니다.
하지만 현재 우리가 달리는 것은 필요한 지식을 습득 한 후 프로젝트 규모에 따른 기술적인 선택을 원활하게 할 수 있는 역량을 키우기 위함도 있습니다. 이런 저런 것 도입해주시고, 제가 의견 계속 드리겠습니다~

p3; 의견으로는 ci 세팅등은 env: 등의 precommit message 가 적절할 것 같아요

고생 많으셨습니다~!

@bytrustu
Copy link
Member Author

좋습니다! husky 같은 경우엔 대형 프로젝트엔 유효할 수 있지만, 소규모 프로젝트엔 병목이 될 수 있다는 생각도 있습니다. 하지만 현재 우리가 달리는 것은 필요한 지식을 습득 한 후 프로젝트 규모에 따른 기술적인 선택을 원활하게 할 수 있는 역량을 키우기 위함도 있습니다. 이런 저런 것 도입해주시고, 제가 의견 계속 드리겠습니다~

p3; 의견으로는 ci 세팅등은 env: 등의 precommit message 가 적절할 것 같아요

고생 많으셨습니다~!

감사합니다! 환경 셋팅 관련 내용은 env: 로 해보겠습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants