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 적용 후 rebase #89

Merged
merged 2 commits into from
Jul 19, 2024
Merged

Conversation

zeus6768
Copy link
Contributor

⚡️ 관련 이슈

#83

📍주요 변경 사항

develop 브랜치에 ci 적용 후 dev/be 브랜치를 rebase 합니다.

🎸기타

dev/be, dev/fe 브랜치만 develop 브랜치로 pr을 보내주세요.

* chore(workflows): 백엔드 pr 생성 시에 빌드하는 워크플로 생성

* refactor(workflows): 백엔드 배포 자동화 워크플로 이름 변경 생성

* chore: gradle build에서 test task 로 변경

* fix: mysql dialect 지정

* fix: mysql 실행 순서 변경

* fix: mysql root password로 수정

* fix: mysql을 setup 해주는 action 라이브러리 변경
@zeus6768 zeus6768 added the BE 백엔드 label Jul 18, 2024
@zeus6768 zeus6768 added this to the 2차 스프린트 🐣 milestone Jul 18, 2024
@zeus6768 zeus6768 requested review from jminkkk and kyum-q July 18, 2024 08:56
@zeus6768 zeus6768 self-assigned this Jul 18, 2024
jminkkk
jminkkk previously approved these changes Jul 18, 2024
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.

수고하셨습니다 ~🔥

kyum-q
kyum-q previously approved these changes Jul 18, 2024
Copy link
Contributor

@kyum-q kyum-q left a comment

Choose a reason for hiding this comment

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

LGTM 👍✨

@zeus6768 zeus6768 dismissed stale reviews from kyum-q and jminkkk via 3f80be1 July 19, 2024 01:45
@zeus6768
Copy link
Contributor Author

실패하는 배포 코드 삭제했어요. 확인하시고 approve 해주세요~

@zeus6768 zeus6768 requested review from kyum-q and jminkkk July 19, 2024 01:47
@zeus6768 zeus6768 changed the title [BE] rebase [BE] CI 적용 후 rebase Jul 19, 2024
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.

수고하셨슴다!

Copy link
Contributor

@kyum-q kyum-q left a comment

Choose a reason for hiding this comment

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

좋아요 굿굿 ~!

@zeus6768 zeus6768 requested review from zangsu, jminkkk and HoeSeong123 and removed request for jminkkk July 19, 2024 05:00
@zeus6768 zeus6768 linked an issue Jul 19, 2024 that may be closed by this pull request
2 tasks
@HoeSeong123 HoeSeong123 changed the title [BE] CI 적용 후 rebase CI 적용 후 rebase Jul 19, 2024
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.

켬미의 위키를 보면서 쭉- 읽어 봤습니다!
파일 내용 자체에 대한 궁금한 점은 없어 Approve 할게요!
고생하셨습니다 ⚡

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.

고생하셨습니다.
어려운 내용이었는데 문서화가 잘 되어 있어서 이해하기 쉬웠네요.

@HoeSeong123 HoeSeong123 merged commit 2e7c172 into woowacourse-teams:dev/be Jul 19, 2024
1 check passed
@zeus6768 zeus6768 deleted the dev/be branch July 31, 2024 04:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드
Projects
Status: Weekend Done
Development

Successfully merging this pull request may close these issues.

[FEAT] CI 파이프라인 구축
5 participants