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

공통으로 사용되는 함수 분리 #27

Open

CODe5753 opened this issue Apr 24, 2023 · 0 comments

Assignees

Labels

enhancement Improvement Priority Low

Member

CODe5753 commented Apr 24, 2023 •

edited

Loading

n번째 기수 영어 표기 함수가 반복적으로 사용되고 있습니다.
사실 이 로직은 앞으로 변경될 여지가 없습니다.
따라서, 함수 분리를 꼭 해야하는지에 대해서 검토가 필요한 부분입니다.

다만, n번째 기수 영어 표기하는 함수와 마찬가지로 코드 전역적으로 사용되는 함수들이 더 있다면, 이를 공통으로 묶어 관리하는 것이 좋아보입니다.

결론은, n번째 기수 영어 표기 함수뿐 아니라 여러 공통 함수들을 묶어서 처리하는 것이 좋은 방향같습니다.

Originally posted by @CODe5753 in #25 (comment)

CODe5753 changed the title ~~n번째 기수 영어 표기 함수 분리~~ 공통으로 사용되는 함수 분리

CODe5753 added enhancement Improvement Priority Low labels

emost22 self-assigned this

emost22 mentioned this issue

fix(recruitment): hide d-day before recruitment #26

Merged

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