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

[PC-333] 깃허브 액션 CICD 설정 with docker #24

Open
wants to merge 12 commits into
base: develope
Choose a base branch
from

Conversation

devchlee12
Copy link
Member

@devchlee12 devchlee12 commented Jan 15, 2025

🔗 관련 이슈

PC-333

✨ 작업 내용

  • develope 브랜치로 PR이 올라가면 테스트를 수행하고, 결과가 표시되도록 하였습니다.
  • 변경 때문에 깨지던 테스트코드를 수정하였습니다.
  • 깃허브 액션 Runner에서 도커 파일로 빌드해서 허브로 올린 뒤 ssh로 실행할 수 있도록 하였습니다.

✅ 체크리스트

  • 코드가 정상적으로 컴파일되나요?
  • 테스트 코드를 통과했나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

🎃 새롭게 알게된 사항

📋 참고 사항

  • 소나 큐브는 좀 더 공부해보고 적용하겠습니다.

Copy link

github-actions bot commented Jan 15, 2025

Unit Test Results

0 tests   0 ✔️  0s ⏱️
0 suites  0 💤
0 files    0

Results for commit e523c0b.

♻️ This comment has been updated with latest results.

@Lujaec
Copy link
Member

Lujaec commented Jan 18, 2025

api 모듈과 admin모듈은 각각 별개의 spirng applicatoin으로 동작해야하는데 고려하셨을까용 ?

api 모듈에서의 배포는 개발환경과 운영환경의 배포는 구분되어야하고 admin 모듈은 운영환경만 있으면 될 것 같습니다 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants