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
Generate the build number at the beginning of the build #6894
Conversation
Pull Request Checklist
|
6c2d199
to
0e19f56
Compare
# Example: 1812011805 | ||
date '+%y%m%d%H%M' | tee "${BUILD_NUMBER_FILE}" | ||
# Format: Year(2 digit) + Month + Day + Hour | ||
# Example: 2018120118 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment is wrong now. Why this change, btw?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've removed minutes to reduce possibility of getting different build numbers for different platforms.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, fixed comment.
@jakubgs I've checked Android and iOS builds from #6894 (comment) and it's good! Both apps has (2018112717) version. Could you please review comments above about comments (sorry for tautoligies! :) inside PR so we could proceed as we need this changes in 0.9.32 release. |
508adbe
to
261e6f6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm now
Signed-off-by: Jakub Sokołowski <jakub@status.im>
261e6f6
to
7e031e8
Compare
Generate the build number at the beginning of the build to avoid mismatched numbers.
Resolves. #6881