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

refactor: Github Actions 워크플로 리팩토링 #230

Merged
merged 9 commits into from
Jan 2, 2024

Conversation

bbearcookie
Copy link
Member

@bbearcookie bbearcookie commented Jan 1, 2024

🧩 이슈 번호

✅ 작업 사항

  • main 브랜치에 push 시 동작하는 yml 파일을 하나로 합쳤어요.
    • (기존에는 Express, React 가 따로따로 분리되어 있다보니 Actions 탭에 동시에 돌아가는게 신경이 쓰였어요.)
  • 자주 반복적으로 사용되는 Node.js 설정 및 Yarn Install을 커스텀 액션으로 작성해두었어요.
    • (사실 Yarn Berry의 Zero-install을 사용하고 있기 때문에, 패키지 인스톨이 없어도 된다고 생각하는데.. 왠지 모르겠지만 React 앱에서는 인스톨 없이 빌드가 안되네요.. 일부 패키지에서 pnp를 지원하지 않아서 그런건지..)
  • swagger-cli 빌드에 validate 커맨드 추가
  • Express 빌드에 테스트 코드 제외

👩‍💻 공유 포인트 및 논의 사항

현재 브랜치에는 아직 Express 앱의 테스트 코드 관련 설정이 되어있지 않기 때문에 CI에서 오류가 발생하는데, 이는 #227 PR이 머지된 이후에는 정상적으로 동작할 것으로 예상해요!

@bbearcookie bbearcookie added the 🔨 Refactor 리팩토링 label Jan 1, 2024
@bbearcookie bbearcookie self-assigned this Jan 1, 2024
@bbearcookie bbearcookie merged commit d557211 into main Jan 2, 2024
2 checks passed
@bbearcookie bbearcookie deleted the refactor/228-github-actions branch January 2, 2024 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Github Actions 워크플로 정리
2 participants