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

Feature/#085 Github Actions workflow 추가 #90

Merged
merged 3 commits into from
Nov 11, 2024

Conversation

minjungw00
Copy link
Collaborator

📝 변경 사항

  • Github Actions 설정 #85
  • PR을 올릴 때마다 자동으로 lint와 테스트를 수행하는 workflow 추가
  • 2명 이상이 approve한 PR을 자동으로 merge하는 workflow 추가

🔍 변경 사항 설명

  • PR 자동 merge workflow의 경우 재도님이 공유하신 글을 참고해서 작성했습니다.

🙏 질문 사항

  • 리뷰어에게 부탁하고싶은 체크리스트를 추가합니다.

📷 스크린샷 (선택)

  • UI 변경이 있는 경우 스크린샷이나 GIF를 첨부합니다.

✅ 작성자 체크리스트

  • Self-review: 코드가 스스로 검토됨
  • Unit tests 추가 또는 수정
  • 로컬에서 모든 기능이 정상 작동함
  • 린터 및 포맷터로 코드 정리됨
  • 의존성 업데이트 확인
  • 문서 업데이트 또는 주석 추가 (필요 시)

@minjungw00 minjungw00 added the Build 개발 환경 세팅(eslint, 컨벤션 등) 또는 패키지 매니저 수정 label Nov 10, 2024
@minjungw00 minjungw00 self-assigned this Nov 10, 2024
Copy link
Collaborator

@Ludovico7 Ludovico7 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
Member

@pipisebastian pipisebastian left a comment

Choose a reason for hiding this comment

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

client config 파일에서 린트에러가 뜨는군요..
이거는 이전 PR(root, client, server config 파일 린트 ignore) 머지되면 해결해주려나요??

@minjungw00
Copy link
Collaborator Author

client config 파일에서 린트에러가 뜨는군요.. 이거는 이전 PR(root, client, server config 파일 린트 ignore) 머지되면 해결해주려나요??

다른 파일들 최대한 안건드리고 workflow 파일만 추가한 브랜치라 lint 적용이 아직 안됐습니다! 머지되고 lint 잡으면 될 것 같아요!

Copy link
Collaborator

@hyonun321 hyonun321 left a comment

Choose a reason for hiding this comment

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

merge 후 린트 확인했습니다! 감사합니다 세팅해주셔서 후후..

@hyonun321 hyonun321 merged commit 318e649 into dev Nov 11, 2024
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build 개발 환경 세팅(eslint, 컨벤션 등) 또는 패키지 매니저 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants