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

[Refactor] build-test.yml 스크립트 수정 #175

Merged
merged 1 commit into from
Sep 16, 2024
Merged

[Refactor] build-test.yml 스크립트 수정 #175

merged 1 commit into from
Sep 16, 2024

Conversation

jayn2u
Copy link
Member

@jayn2u jayn2u commented Sep 15, 2024 •
edited
Loading

🍎 빈주머니즈 iOS Pull Request

✏️ 개요

변경 사항 및 관련 이슈에 대해 간단하게 작성해주세요. 어떻게보다 무엇을 왜 수정했는지 설명해주세요

  • 정상적으로 동작하는 워크플로우로 보완했습니다.
  • qa 브랜치 방향으로 PR이 병합될 때, build-test라는 브랜치로 원격으로 푸시될 때 마다 CI 파이프라인이 동작합니다.

🚨 변경사항

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • 문서 수정
  • 코드 포맷 변경, 세미콜론 누락, 코드 수정이 없는경우
  • 코드 리팩토링
  • 테스트 추가, 테스트 리팩토링
  • 빌드 수정, 패키지 매니저 수정
  • 파일, 폴더명 수정
  • 파일, 폴더 삭제

✅ Checklist

체크리스트를 작성해주세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.
  • import 코드와 같은 불필요한 코드들을 삭제했습니다.
  • 기존의 코드에 영향이 없는 것을 확인했습니다.
  • Apple Swift-Format을 사용해서 코드 포맷을 정리했습니다.

📱 구현화면

gif 혹은 스크린샷 첨부해주세요!

🛠️ issue 연결

  • Resolved : #이슈번호

@jayn2u jayn2u added the 🧰 Refactor 리팩터링이 필요한 내역들 label Sep 15, 2024
@jayn2u jayn2u self-assigned this Sep 15, 2024
SeungEEE
SeungEEE approved these changes Sep 15, 2024
View reviewed changes
Copy link
Member

@SeungEEE SeungEEE left a comment

Choose a reason for hiding this comment

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

확인했습니다!!

@jayn2u jayn2u merged commit 03610de into develop Sep 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SeungEEE SeungEEE SeungEEE approved these changes

@SeungWon1125 SeungWon1125 Awaiting requested review from SeungWon1125

Assignees

@jayn2u jayn2u

Labels
🧰 Refactor 리팩터링이 필요한 내역들
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jayn2u @SeungEEE