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

[refactor] dev 환경이 사라짐에 따른 코드, 설정, ci/cd 수정 #501

Merged
merged 10 commits into from
Apr 5, 2024

Conversation

mooncw
Copy link
Collaborator

@mooncw mooncw commented Apr 5, 2024

PR을 보내기 전에 확인해주세요!!

  • 컨벤션에 맞게 작성했습니다. 컨벤션 확인은 여기
  • 변경 사항에 대한 테스트를 했습니다.

관련 이슈

close: #500

개요

dev 환경이 사라짐에 따른 코드, 설정, ci/cd 수정

상세 내용

  • Jasypt 추가
  • member-api 모듈의 기존 application.yml을 application.yml, application-common.yml, application-local.yml, application-prod1.yml, application-prod2.yml로 분리(application-local.yml는 깃허브에 올리지 않으므로 따로 만드셔야합니다.)
  • 필요없는 application.yml과 application.yml내의 설정들 제거
  • ci/cd 수정 및 cd-dev 삭제
  • ci 테스트할 때 필요한 datahub는 깃허브 액션 환경에 docker-compose로 설치하도록 구성
  • 코드 레벨에서 url 변경, 서비 환경 구분을 위한 코드 수정

@mooncw mooncw added Setting 개발 환경 세팅 Refactor 코드 리팩토링 labels Apr 5, 2024
@mooncw mooncw self-assigned this Apr 5, 2024
Comment on lines -78 to +39
./gradlew clean build
./gradlew build
Copy link
Collaborator

Choose a reason for hiding this comment

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

혹시 왜 바꾸신지 궁금하신 분들을 위한 주석
요약하면, ./gradlew clean build는 이전 빌드의 영향을 받지 않는 깨끗한 상태에서 프로젝트를 빌드하고자 할 때 사용하며, ./gradlew build는 빠른 빌드 시간을 위해 이전 빌드의 결과물을 재사용하고자 할 때 사용

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

이전 빌드 결과물이 없어서요

@mooncw mooncw merged commit d45974a into develop Apr 5, 2024
1 check passed
@mooncw mooncw deleted the refactor/#500-dev-server-remove branch April 5, 2024 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Refactor 코드 리팩토링 Setting 개발 환경 세팅
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[refactor] dev 인스턴스 서버가 사리짐에 따른 리팩토링이 필요한 부분 발생
2 participants