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

fix: migrate db on build & add lint action #24

Merged
merged 3 commits into from
Feb 10, 2023
Merged

fix: migrate db on build & add lint action #24

merged 3 commits into from
Feb 10, 2023

Conversation

aube-dev
Copy link
Member

@aube-dev aube-dev commented Feb 9, 2023

Issue

할당된 이슈 없음

Description

Fork repository에서는 environment secrets에 접근할 수 없어 db migration action이 작동하지 않는 문제가 있었습니다. (#23)
그래서 db migration action들을 모두 삭제하고, 대신 vercel에서 build phase에 db migration을 할 수 있도록 커맨드를 추가했습니다. (package.jsonbuild 참고)
수정 이후에는 모든 PR이 DB migration을 실행하게 됩니다. 따라서 DB 구조를 덮어쓸 가능성이 더 높아지므로, 이런 일을 최소화하기 위해 PR 템플릿의 체크리스트도 조금 수정했습니다.

그리고 lint action을 추가했습니다. lint에서 error에 해당하는 규칙을 어긴 경우 vercel 빌드가 실패하게 되는데 vercel 로그를 볼 수 있는 권한을 제가 일일이 부여할 수 없는 상황입니다. 더불어 lint 규칙을 지키는 것은 프로젝트에서 중요하기도 하므로, action을 따로 추가해서 PR 상에서도 볼 수 있도록 했습니다. lint warning은 빌드를 실패시키지 않아서 넘어가기 쉽지만 이렇게 하면 github actions 로그상에는 뜨게 되니, 코드리뷰할 때 한번씩 확인해 보시고 웬만하면 warning도 뜨지 않도록 수정하면 좋을 것 같아요.

Check List

  • PR 제목을 커밋 규칙에 맞게 작성
  • 적절한 라벨 설정
  • GitHub Projects에 연결
  • 작업한 사람 모두를 Assign
  • Code Review 요청
  • DB Schema를 바꾼 경우, DB Schema를 바꾸는 다른 PR이 없는지 확인
  • main 브랜치의 최신 상태를 반영하고 있는지 확인

@aube-dev aube-dev added bug Something isn't working documentation 주석, README 등 문서화 관련 labels Feb 9, 2023
@aube-dev aube-dev self-assigned this Feb 9, 2023
@vercel
Copy link

vercel bot commented Feb 9, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
web ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 9, 2023 at 7:38PM (UTC)

@aube-dev aube-dev temporarily deployed to Preview February 9, 2023 19:12 — with GitHub Actions Inactive
@aube-dev aube-dev temporarily deployed to Preview February 9, 2023 19:38 — with GitHub Actions Inactive
@aube-dev aube-dev temporarily deployed to Preview February 9, 2023 19:38 — with GitHub Actions Inactive
Copy link
Contributor

@kanghyun98 kanghyun98 left a comment

Choose a reason for hiding this comment

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

🐰 캬.. 정민님 PR에서 현재 오류가 난 부분이 db url을 인식 못하는거 보고 .env 파일을 정상적으로 처리 못하는건가 라고 생각만 했는데.. 이런 문제가 있었던거군여

갓벽한 광서님의 해결방법.. 한 수 배웁니다👍🏼👍🏼

@aube-dev aube-dev added the build 의존성, 스크립트, CI/CD 등 label Feb 10, 2023
Copy link
Member

@JjungminLee JjungminLee left a comment

Choose a reason for hiding this comment

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

🐰 으어 정말 수고하셨어요! 차라리 제가 브랜치를 따로 파서 할걸 그랬나봐요 죄송해요 ㅜㅜㅜ DB 구조를 덮어쓸 가능성이 있다는 점에 염두해서 작업하도록 할게요! 감사합니다!

@aube-dev aube-dev merged commit f5e411f into main Feb 10, 2023
@aube-dev aube-dev deleted the fix/pr-actions branch February 10, 2023 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working build 의존성, 스크립트, CI/CD 등 documentation 주석, README 등 문서화 관련
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants