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

[GGFE-139] husky prepare-commit-msg 추가 #910

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

yoouyeon
Copy link
Member

@yoouyeon yoouyeon commented Aug 1, 2023

📌 개요

  • 배포하면서 사라졌던 husky 설정을 복구하고 지라 이슈 키를 커밋메시지에 포함시키는 스크립트를 추가했습니다.

💻 작업사항

prepare-commit-msg 내용

  • main, dev,(저흰 dev 브랜치가 없지만 혹시 몰라 추가해뒀습니다), deploy를 제외한 브랜치명은 GGFE-숫자 로 시작해야 합니다. (지라에서 브랜치 만들면 규칙대로 만들어집니다)
  • [Feat] [issue_key] commit message 형식으로 커밋 메시지를 만들어줍니다.
  • 만약에 커밋 메시지에 이슈 키가 있으면 브랜치명의 이슈 키와 비교해서 동일하면 바로 커밋하고, 아닌 경우에는 에러를 띄웁니다.

✅ 변경로직

- 브랜치명의 jira issue key 커밋 메시지에 형식에 맞춰 추가하기
@yoouyeon yoouyeon self-assigned this Aug 1, 2023
Copy link
Contributor

@PHJoon PHJoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

제가 제대로 테스트해본건진 모르겠는데 커밋 메세지에 [GGFE-1a] 나 [GGFE-aa] 같이 숫자 아닌 문자 잘못 넣으면 커밋 되는거 같아서
if [[ -z $issue_key_from_commit_msg ]]; then echo "커밋 메시지의 이슈 키가 브랜치명의 이슈 키와 일치하지 않습니다." exit 1 fi
가 추가되면 될거같아요. 다른 경우는 다 잘되는 듯 합니다. 고생하셨습니다! 👍

@yoouyeon
Copy link
Member Author

yoouyeon commented Aug 2, 2023

-z 옵션을 넣어주게 되면 없는 경우에 이슈 키를 넣지 못하고 에러가 나서 그냥 그대로 두겠습니다..!

@yoouyeon yoouyeon merged commit 8af1d34 into main Aug 2, 2023
@yoouyeon yoouyeon deleted the GGFE-139-husky-rule-수정 branch August 2, 2023 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants