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

CI/CD #3 - CI/CD 구축 #4

Merged
merged 7 commits into from
Jun 14, 2024
Merged

Conversation

DongwookKim0823
Copy link
Member

@DongwookKim0823 DongwookKim0823 commented Jun 12, 2024

PR 체크리스트

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

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

PR 유형

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

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

관련 이슈

이슈 번호: #3

어떤 동작은 무엇인가요?

  • Github Actions CI/CD 설정

기타 정보

- github actions용 ci 스크립트 생성
- github actions용 cd 스크립트 생성
Resolves #3
@JaeHyuckSa JaeHyuckSa changed the title DEPLOY #3 - CI/CD 구축 Deploy #3 - CI/CD 구축 Jun 12, 2024
@JaeHyuckSa JaeHyuckSa changed the title Deploy #3 - CI/CD 구축 CI/CD #3 - CI/CD 구축 Jun 12, 2024
@JaeHyuckSa
Copy link
Member

JaeHyuckSa commented Jun 12, 2024

@DongwookKim0823 PR 제목 작성해주실 때
https://hiallen.notion.site/PR-Issue-Bug-Convention-7f02a8337ea0441689f63be2d4c1ce71?pvs=4
참고해주시면 됩니다!

- github actions에서 pre-commit 검사 실패로 인해 수정
@JaeHyuckSa
Copy link
Member

고생하셨습니다

- pre-commit 설정으로 포매팅, 린팅 체크 방식에서 기존 방식으로 수정
- docker compose 실행 명령어 수정
- 에러 원인을 찾기 위해 DJANGO_SETTINGS_MODULE 주석 처리
- guest_service 관련 설정 수정
@DongwookKim0823 DongwookKim0823 merged commit 52ddef3 into develop Jun 14, 2024
1 check passed
@DongwookKim0823 DongwookKim0823 deleted the deploy/mung-83/ci-cd-setting branch June 14, 2024 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌏 Deploy 배포 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants