-
Notifications
You must be signed in to change notification settings - Fork 0
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
Be/refactor/#582 로거 공통 패키지로 분리 #583
Merged
The head ref may contain hidden characters: "BE/refactor/#582-\uB85C\uAC70-\uACF5\uD1B5-\uD328\uD0A4\uC9C0\uB85C-\uBD84\uB9AC"
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
workspaces 속성을 이용하여 여러 패키지를 관리하도록 리팩토링
- was, signal 서비스에서 같은 로거를 사용 - packages 하위에 로거를 정의하고 사용하도록 수정
- was와 signal 서비스에서 같은 로거를 사용함 - 같은 코드가 서로 다른 위치에 중복으로 정의되었으므로 공통 라이브러리로 분리
최상위 디렉토리에서 설치했으므로 하위 디렉토리에 또 다시 설치할 필요 없음
- 공통 패키지와 관련된 내용은 최상위 package.json에 기록되어 있음 - 해당 파일을 app 하위에 복사하여 이미지가 코드를 찾을 수 있도록 수정
최상위 디렉토리에서 공용 패키지를 빌드하고, dev를 실행할 수 있도록 스크립트 작성
HeoJiye
approved these changes
Feb 28, 2024
오오... 프론트도 나중에 이런부분 패키지로 따로 빼면 좋겠다👍 |
Doosies
approved these changes
Feb 28, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔮 resolved #582
변경 사항
로거 관련 코드 공통 패키지로 분리
고민과 해결 과정
was
와signal
서비스는 백엔드 서비스지만 별도의 프로젝트로 분리되어있다.was
와signal
모두 오류 추적 및 유지보수를 위해 로거를 사용하고 있지만, 공통 패키지를 사용하지 않아 코드를 중복해서 사용하고 있다.이를 해결하기 위해 로거를 별도의 프로젝트로 분리하고, 이를
was
,signal
서비스에서 재사용하는 방식으로 구현했다.1️⃣
workspaces
workspaces
는 npm 버전7 이상에서 사용할 수 있는 기능으로, 여러 패키지를 하나의 최상위 패키지에서 관리하는 기능을 제공한다.2️⃣ 공통 패키지 작성
공통 패키지들은
packages
하위에 저장된다. 로거 코드를 옮긴 후package.json
을 작성하여 빌드 결과물을 export 한다.Note
공통적으로 사용되는 라이브러리들을 최상위 디렉토리로 빼서 다시 install 해줘야 해!
최상위 디렉토리에서 아래 명령어 실행하고, was, signal, frontend 다시 install 하면 돼
(선택) 테스트 결과