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.

Sign up for GitHub

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

Jump to bottom

[Refactor/316] TagGenerator 리팩토링 #317

Merged

hosung-222 merged 2 commits into dev from refactor/316

Sep 20, 2024

Member

hosunglee222 commented Sep 19, 2024

Type of change

Feature : 새로운 기능 추가
Bug fix : 버그 수정
Refactor : 코드 리팩토링 작업
Document : 문서작업
Test : 테스트 코드 작성 및 테스트 작업
Style : 코드 스타일 및 포맷팅 작업
CI/CD : CI/CD 작업 수정
Chore : 패키지 매니저, 라이브러리 업데이트 등의 작업

PR Desciption

변경 사항 설명

태그를 발급하는 각 로직을 분리하고, 상수를 활용해 유지보수성 향상
- 태그 생성하는 generateRandomTag()
- 태그 중복 검증하는 generateUniqueTag()

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

첨부 자료

관련 이슈

close [Refactor] TagGenerator 리팩토링 #316


          ♻️ Refactor: TagGenerator 리팩토링

4fe95e3

<body>
태그를 발급하는 각 로직을 분리하고, 상수를 활용해 유지보수성 향상

- 관련 : #313

hosunglee222 added the ♻️ refactor label

hosunglee222 requested a review from joowojr

September 19, 2024 09:01

hosunglee222 self-assigned this

hosunglee222 linked an issue

that may be closed by this pull request

[Refactor] TagGenerator 리팩토링 #316

Closed

10 tasks


          ♻️ Refactor: 사용안하는 import 제거

6f8a181

<body>
미사용 import 제거

- 관련 : #313

hosung-222 merged commit de3b923 into dev

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

Labels

♻️ refactor