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.

Sign up for GitHub

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

Jump to bottom

[FIX] CI/CD 스크립트 위치 수정 #26

Merged
merged 4 commits into from
Jul 18, 2024
Merged

[FIX] CI/CD 스크립트 위치 수정 #26

merged 4 commits into from
Jul 18, 2024

Conversation

PgmJun
Copy link
Member

@PgmJun PgmJun commented Jul 18, 2024 •
edited
Loading

Issue Number

#22

As-Is

CI/CD 스크립트가 최상위 패키지에 존재하지 않고 backend 패키지에 존재하고 있었다.

To-Be

  • CI/CD 스크립트를 최상위 패키지로 이동시켰다.
  • 스크립트의 working directory를 최상위 패키지에서 ./backend로 변경했다.

Check List

  • 테스트가 전부 통과되었나요?
  • 모든 commit이 push 되었나요?
  • merge할 branch를 확인했나요?
  • Assignee를 지정했나요?
  • Label을 지정했나요?

Test Screenshot

(Optional) Additional Description

@PgmJun PgmJun added 🛠 fix 버그 🍃 BE back end labels Jul 18, 2024
@PgmJun PgmJun self-assigned this Jul 18, 2024
jhon3242
jhon3242 approved these changes Jul 18, 2024
View reviewed changes
Copy link
Member

@jhon3242 jhon3242 left a comment

Choose a reason for hiding this comment

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

든든 칭찬해~

@PgmJun PgmJun merged commit c46c268 into develop Jul 18, 2024
1 check passed
@PgmJun PgmJun deleted the feat/#22 branch July 18, 2024 01:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jhon3242 jhon3242 jhon3242 approved these changes

@GIVEN53 GIVEN53 GIVEN53 approved these changes

@leegwichan leegwichan Awaiting requested review from leegwichan

Assignees

@PgmJun PgmJun

Labels
🍃 BE back end 🛠 fix 버그
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@PgmJun @jhon3242 @GIVEN53