PLUS-POSTECH / soma

Cross-platform CTF problem container manager
Apache License 2.0
24 stars 3 forks source link

Bump up package version and remove serde_derive #24

Closed KSAlpha closed 5 years ago

Qwaz commented 5 years ago

혹시 MAJOR MINOR PATCH 버전 붙은게 있고 안 붙은게 있는데 각각 차이가 뭔가요?

KSAlpha commented 5 years ago

저게 기억하기로는 "2" 면 major 버전이 2인것 중에 최신 "0.1" 이면 major 버전이 0이고 minor 버전이 1이상인 것 중에 최신인데

보통은 api change가 일어나지 않는 선까지 자르는걸로 알고 있습니다.

Qwaz commented 5 years ago

저도 그렇게 쓰면 무슨 버전을 찾게 되는지는 아는데, 강수님이 major로 자른거랑 minor로 자른거랑 patch로 자른 기준을 알려달라는 의미였습니다.

KSAlpha commented 5 years ago

Minor로 자르는게 디폴트고, doc에서 다르게 자르면 맞춰서 짤랐습니다

Qwaz commented 5 years ago

Cargo dependency 디폴트가 caret requirements인데요, minor에서 자르면 정보 손실이 생기기 때문에 저는 가능하면 patch 버전까지 명시하는게 낫다고 생각합니다. 강수님 생각은 어떠신가요?

^1.2.3 := >=1.2.3 <2.0.0
^1.2 := >=1.2.0 <2.0.0
KSAlpha commented 5 years ago

기본이 caret requirement라는 사실은 몰랐네요... 제안해주신대로 그냥 patch version까지 다 적는 방향으로 하는게 좋을 것 같습니다.

KSAlpha commented 5 years ago

충돌 가능성이 높아서 #28 이후로 머지하겠습니다.

Rebase onto master: https://github.com/PLUS-POSTECH/soma/pull/24/commits/6ba09405e6a3d9af016c20ea85e7795727e7a865