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

[#47] Feature : CI 구축 #48

Merged
merged 16 commits into from
Jan 15, 2023
Merged

[#47] Feature : CI 구축 #48

merged 16 commits into from
Jan 15, 2023

Conversation

devKangMinHyeok
Copy link
Contributor

@devKangMinHyeok devKangMinHyeok commented Jan 12, 2023

[#47] Feature : CI 구축

테스트 자동화를 위한 CI 구축


CHANGES

폴더 구조 재구성
ESLint 적용
Prettier 적용
CI yml 스크립트 작성
test Code 파일 작성
build 전, merge 막기 설정
babel 설정


TEST

다운받기
pull => npm i

테스트

  • 테스트 케이스 확인 후 의도적으로 테스트 케이스 바꿔보면서 테스트 실행 환경 테스팅 가능
  • npm run build : dist 파일제거 후 tsc로 새로운 dist 파일 생성 및 ts 컴파일 빌드
  • npm run lint : eslint 실행
  • npm run test : jest 실행
  • npm run pre-test: eslint, jest 순차적 실행 (매번 작업 후 commit 전 pre 커맨드 돌려보며 확인)

예시

  • 의도적으로 example6.s를 수정하여 test 실패하게 만든 경우

image

@devKangMinHyeok devKangMinHyeok changed the title [#47] Feature : CI 구축 [#47] [아직 작업 중입니다~!]Feature : CI 구축 Jan 12, 2023
@devKangMinHyeok devKangMinHyeok changed the title [#47] [아직 작업 중입니다~!]Feature : CI 구축 [#47] Feature : CI 구축 Jan 13, 2023
@bams11
Copy link
Contributor

bams11 commented Jan 13, 2023

테스트 완료

  • npm run build : .dist 삭제 되고 다시 빌드 되는 것 확인
  • npm run lint : 의도적으로 코드에 오타 내서 eslint가 잘 감지하는지 확인
  • npm run test : sample_output 에 있는 예시 변경하여 test가 잘 이루어지는지 확인
  • npm run pre-test: 위와 같은 방법으로 test가 잘 이루어지는지 확인

image

@haejunejung
Copy link
Contributor

테스트 완료

✓ npm run build : dist 파일제거 후 tsc로 새로운 dist 파일 생성 및 ts 컴파일 빌드
✓ npm run lint : eslint 실행
✓ npm run test : jest 실행
✓ npm run pre-test: eslint, jest 순차적 실행

@limetree06
Copy link
Contributor

테스트 완료

✓ npm run build : dist 파일제거 되고 컴파일된 js 파일 생성되는지 확인 완료
✓ npm run lint : eslint 실행 확인완료
✓ npm run test : jest 실행되며 test 잘 이루어지는것 확인 완료
✓ npm run pre-test: 위 과정 순차적으로 실행되는지 확인 완료

@limetree06
Copy link
Contributor

확인완료했습니다 머지할께요

@limetree06 limetree06 merged commit 14ad182 into main Jan 15, 2023
@limetree06 limetree06 deleted the feature/47/CI_making branch January 15, 2023 11:59
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.

None yet

4 participants