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

프로젝트 root 분리 #25

Merged
merged 7 commits into from
Oct 31, 2023
Merged

프로젝트 root 분리 #25

merged 7 commits into from
Oct 31, 2023

Conversation

Chestnut90
Copy link
Collaborator

PR 체크리스트

아래 항목을 확인해 주세요:

  • 커밋 메시지가 우리의 가이드라인을 따르고 있는지 확인하세요
  • 변경 사항에 대한 테스트가 추가되었는지 확인하세요 (버그 수정 / 기능 추가)
  • 문서가 추가되거나 업데이트되었는지 확인하세요 (버그 수정 / 기능 추가)

PR 유형

이 PR은 어떤 종류의 변경을 가져오나요?

  • 버그 수정
  • 새로운 기능 추가
  • 코드 스타일 업데이트 (서식, 로컬 변수)
  • 리팩터링 (기능 변경 없음, API 변경 없음)
  • 빌드 관련 변경
  • CI 관련 변경
  • 문서 내용 변경
  • 애플리케이션 / 인프라 변경
  • 기타... 설명:

현재 동작은 무엇인가요?

이슈 번호: N/A

새로운 동작은 무엇인가요?

이 PR은 호환성 변경을 도입하나요?

  • 아니요

기타 정보

- 개발 환경 구성을 위한 project root와 repository root 분리
- repository root : git, docker, readme 등 환경 및 개발 구성 파일 포함
- project root(src) : 장고 project 포함
- repository root와 project root가 분리 되면서 Docker 환경 세팅
- Dockerfile에서 /app/src/에 project를 프로젝트를 마운트하여 사용
- /app/pyproject.toml은 Poetry venv 활성화하기 위한 조건
@Chestnut90 Chestnut90 self-assigned this Oct 30, 2023
@Chestnut90 Chestnut90 changed the title Feature/repo directory 프로젝트 root 분리 Oct 30, 2023
- flake8의 max-line-length, exclue에 대한 setup.cfg 파일 copy 적용
@JaeHyuckSa
Copy link
Contributor

JaeHyuckSa commented Oct 30, 2023

해당 프로젝트 구조가 더 깔끔하고 더 좋은 것 같습니다! 고생하셨습니다 :)

Copy link
Collaborator

@simseulnyang simseulnyang left a comment

Choose a reason for hiding this comment

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

저는 이 부분에 대한 공부가 필요해서 코드가 어떻게 작성되었는지만 확인했습니다.
다양한 방법이 있다는 사실을 깨달았습니다! 또 배웠네요 감사합니다!😀
그리고 작업 하시느라 수고 많으셨어요!

@Chestnut90 Chestnut90 merged commit e20808c into develop Oct 31, 2023
1 check passed
@JaeHyuckSa JaeHyuckSa deleted the feature/repo-directory branch November 19, 2023 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants