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

의존성 파일 테스트 추가 #37

Merged
merged 1 commit into from
Jul 17, 2019
Merged

Conversation

winterjung
Copy link
Contributor

requirements-dev.txtPipfile의 테스트를 추가했습니다.
각 의존성이 특정한 버전으로 명시되어있는지 검증합니다. (==가 있는지 검사)
일단 저희가 ^, ~, >=등의 specifier는 사용하지 않기에 제외했고 사용하게 될 때 테스트가 터질테니 그 때 테스트 케이스를 수정하면 될 듯 합니다

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.

LGTM

@winterjung winterjung merged commit e0da07c into master Jul 17, 2019
@winterjung winterjung deleted the feature/package-test branch July 17, 2019 04:28
@sean-ahn
Copy link

  1. Pipfiletoml package 인 만큼 toml library 사용해서 파싱하는 것이 더 좋을 듯 합니다.
  2. 버전이 명시되어 있다"*" 이 아니다라고 정의하면 말씀하신 specifier 도 포함할 수 있을 듯 하네요

@winterjung
Copy link
Contributor Author

@sean-ahn toml 라이브러리를 설치하기 귀찮아서 일단 요렇게 했습니당... yaml도 그렇고 기본으로 포함해서 지원하면 좋을텐데 ㅠㅠ PR은 언제나 환영합니다.
"*"도 아직은 고려 안했는데 나중에 언젠가 고려할때가 올거같긴하네요

@sean-ahn
Copy link

@jungwinter Pipfile 은 공식적으로 * 로 두길 권장하고 있고 pipenv sync command 를 사용하길 바라는 것 같아요 install 보다

@winterjung
Copy link
Contributor Author

@sean-ahn 원래라면 석찬님 말대로 Pipfile"*"로 두고 Pipfile.lock으로 관리하는게 맞겠으나 hash관리의 어려움으로 제외했고 차선책으로 Pipfile에 명시하는 방식으로 선회하긴했습니다. (@sunghyunzz 님이랑 구두로만 얘기하고 코멘트로 달지는 못했네용. 관련 이슈 #7 #12 #32 #34)

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.

3 participants