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

⚙️ CI 파이프라인 및 husky 스크립트 수정 #13

Merged
merged 3 commits into from
Apr 23, 2024

Conversation

BangDori
Copy link
Collaborator

@BangDori BangDori commented Apr 23, 2024

작업 이유

ESLint가 Pre-commit 단계로 이동하게 되면서, 기존의 CI 파이프라인에서 테스트하였던 lint 제거


작업 사항

  1. husky 스크립트 수정
  2. CI 파이프라인 스크립트 및 이미지 수정

리뷰어가 중점적으로 확인해야 하는 부분

  • husky 스크립트가 올바르게 수정되었다.
  • CI 파이프라인 이미지가 올바르게 수정되었다.
  • CI 파이프라인 스크립트가 올바르게 수정되었다.

발견한 이슈

  • 없음

@BangDori BangDori self-assigned this Apr 23, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-13.d37mn03xh3qyyz.amplifyapp.com

Copy link
Member

@Legitgoons Legitgoons left a comment

Choose a reason for hiding this comment

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

적절하게 잘 수정하셨습니다. 병합해도 좋습니다. 👍

@BangDori BangDori merged commit a732e69 into main Apr 23, 2024
2 checks passed
@BangDori BangDori deleted the fix/docs-and-script branch April 23, 2024 13:20
Legitgoons pushed a commit that referenced this pull request Apr 25, 2024
- husky install 수정
- CI 파이프라인 스크립트 및 이미지 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants