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에서는 h2 사용으로 변경 #256

Merged
merged 3 commits into from
Aug 5, 2024

Conversation

kyum-q
Copy link
Contributor

@kyum-q kyum-q commented Aug 3, 2024

⚡️ 관련 이슈

close #246

📍주요 변경 사항

CI에서 MySQL 사용을 h2 사용하도록 변경

AS-IS

MySQL 사용

TO-BE

h2 사용으로 변경

🤔 기한

2024-08-03

🎸기타

이슈에 왜 h2로 변경하는지 간략하게 적어놨습니다.
궁금하시다면 이슈를 확인해주세요.

이전 PR에 커밋 문제가 발생해서 close하고 다시 요청보냅니다

@kyum-q kyum-q added refactor 요구사항이 바뀌지 않은 변경사항 BE 백엔드 labels Aug 3, 2024
@kyum-q kyum-q added this to the 3차 스프린트 🐤 milestone Aug 3, 2024
@kyum-q kyum-q self-assigned this Aug 3, 2024
Copy link
Contributor

@HoeSeong123 HoeSeong123 left a comment

Choose a reason for hiding this comment

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

언제나 작업이 빠르네요 켬미..
고생하셨습니다.

Copy link
Contributor

@zangsu zangsu left a comment

Choose a reason for hiding this comment

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

CI 작업은 지속적인 통합을 위해 해당 작업 이후에 test가 잘 작동하는지, 빌드가 되는지를 확인하는 작업이기 때문에 mysql을 사용해서 배포 환경과 비슷하게 해야할 필요성을 모르겠다.

배포 환경과 테스트 환경이 동일하면 어떤 이점이 있는지 / 다르면 어떤 상황이 발생하는지 잘 몰라서요...!
내일 시간 날 때 다시 설명 듣고, approve 남기겠습니다...!

Copy link
Contributor

@zangsu zangsu left a comment

Choose a reason for hiding this comment

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

아니 comment 로 남기려고 했는데;;

Copy link
Contributor

@jminkkk jminkkk left a comment

Choose a reason for hiding this comment

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

수고했어요 켬미!

@jminkkk jminkkk merged commit 5710c3f into woowacourse-teams:dev/be Aug 5, 2024
4 checks passed
@kyum-q kyum-q deleted the refactor/ci-change-h2 branch August 5, 2024 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 refactor 요구사항이 바뀌지 않은 변경사항
Projects
Status: Weekend Done
Development

Successfully merging this pull request may close these issues.

5 participants