Skip to content

[Chanyeol] Week4 미션#26

Open
coconutcococode wants to merge 7 commits intomainfrom
Chanyeol-week4
Open

[Chanyeol] Week4 미션#26
coconutcococode wants to merge 7 commits intomainfrom
Chanyeol-week4

Conversation

@coconutcococode
Copy link
Copy Markdown

🔗 연관 이슈

#13

🛠 작업 내용

도메인형 아키텍처 형태로 member, mission, ask, review 도메인으로 나누어 프로젝트 세팅

🖼 스크린샷 (선택)

👀 리뷰 요구사항 (선택)

🤖 AI 활용

  • AI 사용 안 함
  • 코드 작성 아이디어 참고
  • 테스트/리팩토링 보조
  • 문서/주석 작성 보조
  • 기타 (아래에 간단히 작성)

💬 나의 프롬프트

  1. member 파일 구조 참고해서 같은 방식으로 mission, ask, review 파일 구조 보여줘
  2. 프로젝트 파일들이 다 루트에 올라가 있는데, 파일 옮기는 것 외에 더 해야할 것들 알려줘

🧠 AI 응답

  1. mission, ask, review 파일 구조 제시
  2. 서브모듈 및 불필요한 캐시 파일들 제거 명령어 제시

✅ 내가 최종 선택한 방법 (이유)

  1. 그대로 쓰면 ERD에 벗어나기 때문에 ERD 참고하여 enums, entity 수정

💡 나만의 Tip (선택)

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4c59ccafcd

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@@ -0,0 +1,3 @@
DB_USER=bcy1234
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge .env의 DB 자격증명 커밋을 제거하세요

.envDB_USER/DB_PW가 포함된 상태로 커밋되면 저장소 접근자 누구나 접속 정보를 확인할 수 있고, Git 이력에 남아 비밀번호를 바꿔도 노출 흔적이 유지됩니다. Spring 학습 관점에서도 설정을 저장소에 고정하는 나쁜 습관이 생기므로 이 파일은 추적에서 제거(git rm --cached .env)하고, 샘플은 .env.example로 분리한 뒤 application.yml에서 ${DB_USER}처럼 외부 설정으로 바인딩하세요; 다음 학습 주제로 Externalized Configuration@ConfigurationProperties를 추천합니다.

Useful? React with 👍 / 👎.

!**/src/test/**/build/

### 민감정보 ###
Chanyeol/.env
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge .gitignore의 .env 경로 패턴을 수정하세요

Chanyeol/.gitignore 내부에서 Chanyeol/.env를 쓰면 기준 경로가 이미 Chanyeol/라 실제 .env가 매칭되지 않아 민감 파일이 계속 추적됩니다(이번 커밋에서도 .env가 함께 추가됨). .env 또는 /.env로 패턴을 바꾸고 이미 추적된 파일은 캐시에서 제거해야 같은 실수를 막을 수 있습니다; 다음으로 Git ignore의 상대 경로 규칙을 정리해 두면 좋습니다.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown

@Joonseok-Lee Joonseok-Lee left a comment

Choose a reason for hiding this comment

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

LGTM 👍

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.

[Feat] 4주차-Chapter 4. 프로젝트 세팅하기 - 아키텍처 구조, Swagger

2 participants