Skip to content
This repository has been archived by the owner on Jan 22, 2024. It is now read-only.

Makefile, travis 개선 #34

Merged
merged 6 commits into from
Jun 14, 2019
Merged

Makefile, travis 개선 #34

merged 6 commits into from
Jun 14, 2019

Conversation

winterjung
Copy link
Contributor

@winterjung winterjung commented Jun 14, 2019

Resolve #32

  • Makefilerequirements를 추가했습니다.
    • pipenv을 사용하지 않는 개발자와 협업할 경우 requirements.txt 등의 별도 파일이 필요한데 이를 생성하는 명령어입니다.
  • 커버리지 리포트를 제대로 생성해 codecov에서 사용합니다.
    • pytest --cov-report xml 등이 없을 경우 .travis.yml:after_success의 codecov가 리포트를 인식하지 못합니다.
  • .travis.ymldist 형태를 변경했습니다.
    • 기존 형태는 적합한 yaml 문법이긴 하나 vscode와 IntelliJ에서 Value is not accepted. Valid values: "precise", "trusty", "xenial" 오류를 발생시킵니다.
  • 테스트를 강화했습니다.
    • Makefile, .travis.yml에서 라인 수 뿐만 아니라 덩어리(section)가 몇 개인지도 테스트합니다.
  • Pipfile에 버전을 명시했습니다.
    • Pipfile.lock으로 관리하기엔 비용이 많이 들거라 예상되어 Pipfile에 명시해줬습니다.

@winterjung winterjung self-assigned this Jun 14, 2019
@winterjung winterjung changed the title Feature/makefile Makefile, travis 개선 Jun 14, 2019
@winterjung winterjung requested a review from 0xd669 June 14, 2019 05:24
Copy link
Contributor

@0xd669 0xd669 left a comment

Choose a reason for hiding this comment

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

L.G.T.M 👍

@winterjung winterjung merged commit e3e7877 into master Jun 14, 2019
@winterjung winterjung deleted the feature/makefile branch June 14, 2019 09:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

Makefile 업데이트
3 participants