Skip to content

버전 코드 계산 방식 변경 #443

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

Merged
merged 7 commits into from
Jun 22, 2025
Merged

Conversation

plgafhd
Copy link
Collaborator

@plgafhd plgafhd commented Jun 21, 2025

원래 방식

  • Aa.Bb.Cc -> 2010AaBbCc

바뀐 방식

  • Aa.Bb.Cc-rc.Nn -> 20(A+1)aBbCcNn
  • NN의 default는 99이므로, rc는 98까지 가능

@plgafhd plgafhd requested a review from a team as a code owner June 21, 2025 06:39
@plgafhd
Copy link
Collaborator Author

plgafhd commented Jun 21, 2025

32bit가 2147..까지 가능해서 major version 47까지 커버 가능

@JuTaK97
Copy link
Collaborator

JuTaK97 commented Jun 21, 2025

좀 더 아껴서
aa.bb.cc-rc.NN -> 20(a+1)abbccNN

이건 어때

@JuTaK97
Copy link
Collaborator

JuTaK97 commented Jun 21, 2025

aA.bB.cC-rc.nN -> 20(a+1)AbBcCnN

@JuTaK97
Copy link
Collaborator

JuTaK97 commented Jun 21, 2025

ㅋㅋㅋ 이거 예전에는 완전 나이브하게 돼 있어서 major 3 올리니까 30억 돼서 APK 업로드 안 됐던거 생각나네

@plgafhd
Copy link
Collaborator Author

plgafhd commented Jun 21, 2025

아 이거 표현이 좀 이상하긴 한데 저 aa라는게
예를 들어 3.9.2면
aa = 03 bb = 09 cc = 02
이렇게 생각하고 적었어
그래서 3.9.2에 적용하면
3.9.2-rc.1 -> 2103090201
3.9.2 -> 2103090299

아마 다음 버전도 minor 10 될거고... 모든 파트가 10 이상이 되는걸 고려하고 싶었어

@plgafhd
Copy link
Collaborator Author

plgafhd commented Jun 21, 2025

The greatest value Google Play allows for versionCode is 2100000000.

아 헐 근데 21억 안된다네
아으

@plgafhd
Copy link
Collaborator Author

plgafhd commented Jun 21, 2025

aA.bB.cC-rc.nN -> 20(a+1)AbBcCnN

그냥 이렇게 해야겠다

@JuTaK97
Copy link
Collaborator

JuTaK97 commented Jun 21, 2025

단위테스트 한 3개만 짜놓으면 어떰
주석보다 그게 더 이게 뭔로직인지 알기 쉬울듯

@JuTaK97
Copy link
Collaborator

JuTaK97 commented Jun 21, 2025

로직 수정 아직 안한거지?

@plgafhd
Copy link
Collaborator Author

plgafhd commented Jun 21, 2025

이거 지금이 이 상태 (aA.bB.cC-rc.nN -> 20(a+1)AbBcCnN)야
테스트만 아직 안짰어

@JuTaK97
Copy link
Collaborator

JuTaK97 commented Jun 21, 2025

아맞네 ㅋㅋ

@plgafhd
Copy link
Collaborator Author

plgafhd commented Jun 21, 2025

유닛 테스트 추가까지 완료..!

@JuTaK97
Copy link
Collaborator

JuTaK97 commented Jun 22, 2025

어 근데 RC 없을 때 00으로 안하고 99로 한 이유가 있음?

@plgafhd
Copy link
Collaborator Author

plgafhd commented Jun 22, 2025

아 근데 이거는 rc도 필요할 때는 play console에 올릴거냐에 따라 다르긴 할 것 같은데

내가 rc까지 버전 코드에 넣은건 rc 있는것도 필요할 때는 play console에 올리는 경우를 생각했어
그러면 rc 있는거 -> 없는거 순서일거라 그냥 max 값으로 넣었고

사실 rc 올리는 경우가 흔하진 않을텐데
sdk 대응 잘 됐는지나 배포 관련 자동화 하고 싶으면 있으면 좋겠다 싶겠더라구
요거는 rc까지 play console에 올릴 필요가 없다 싶으면 그냥 0으로 둘게

@JuTaK97
Copy link
Collaborator

JuTaK97 commented Jun 22, 2025

아 근데 이거는 rc도 필요할 때는 play console에 올릴거냐에 따라 다르긴 할 것 같은데

내가 rc까지 버전 코드에 넣은건 rc 있는것도 필요할 때는 play console에 올리는 경우를 생각했어 그러면 rc 있는거 -> 없는거 순서일거라 그냥 max 값으로 넣었고

사실 rc 올리는 경우가 흔하진 않을텐데 sdk 대응 잘 됐는지나 배포 관련 자동화 하고 싶으면 있으면 좋겠다 싶겠더라구 요거는 rc까지 play console에 올릴 필요가 없다 싶으면 그냥 0으로 둘게

아 그렇구나
그럼 99로 하는 것도 괜찮을듯??

@plgafhd
Copy link
Collaborator Author

plgafhd commented Jun 22, 2025

@JuTaK97 형 그러면 다 확인된거면 approve 가능할까??
그 sdk 대응때문에 aab 업로드 해보고 있어서

Copy link
Collaborator

@JuTaK97 JuTaK97 left a comment

Choose a reason for hiding this comment

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

고고~~

@plgafhd plgafhd enabled auto-merge (squash) June 22, 2025 16:02
@plgafhd plgafhd merged commit f803cde into develop Jun 22, 2025
3 checks passed
@plgafhd plgafhd deleted the plgafhd/change-semver-code branch June 22, 2025 16:12
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