Skip to content

배포 스크립트 실행 경로 및 docker-compose 파일 참조 수정#148

Merged
ckdals4600 merged 1 commit intomainfrom
feature/#147-docker-compose-error
Dec 18, 2025
Merged

배포 스크립트 실행 경로 및 docker-compose 파일 참조 수정#148
ckdals4600 merged 1 commit intomainfrom
feature/#147-docker-compose-error

Conversation

@ckdals4600
Copy link
Contributor

관련 이슈

PR 설명

1. 경로 참조 로직 개선

  • 절대 경로 설정: BASH_SOURCEdirname을 사용하여 스크립트가 위치한 디렉토리를 기준으로 REPO_ROOTCOMPOSE_FILE 경로를 동적으로 설정함.
    • 실행 위치(Root vs docker 폴더 등)와 관계없이 항상 올바른 설정 파일을 참조하도록 수정함.
  • 파일 존재 확인: docker-compose.yml 파일이 지정된 경로에 실제로 존재하는지 확인하는 방어 로직을 추가함.

2. 스크립트 리팩토링

  • 명령어 변수화: 반복적으로 사용되는 sudo docker compose -p ... -f ... 구문을 ${COMPOSE} 변수로 정의하여 코드 중복을 제거함.
  • 명령어 교체: 기존의 하드코딩된 docker compose 명령어들을 모두 ${COMPOSE} 변수로 교체하여 유지보수성을 높임.

@ckdals4600 ckdals4600 requested review from Goder-0 and minibr December 18, 2025 12:04
@ckdals4600 ckdals4600 self-assigned this Dec 18, 2025
@ckdals4600 ckdals4600 linked an issue Dec 18, 2025 that may be closed by this pull request
@github-actions
Copy link

📊 코드 커버리지 리포트

Overall Project 91.83% 🍏

There is no coverage information present for the Files changed

@ckdals4600 ckdals4600 merged commit 9cc436e into main Dec 18, 2025
1 check passed
@ckdals4600 ckdals4600 deleted the feature/#147-docker-compose-error branch December 18, 2025 12: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.

Docker compose 설정 변경 미반영 문제

2 participants