-
Notifications
You must be signed in to change notification settings - Fork 120
[Tooling] Use ReleasesV2 version as the source of truth during Code Freeze #16296
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
[Tooling] Use ReleasesV2 version as the source of truth during Code Freeze #16296
Conversation
0f648e8 to
16ed476
Compare
4f7ba8e to
51a65b4
Compare
1cc7d05 to
8a4865e
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.
fastlane/Fastfile
Outdated
| version_short ||= VERSION_FILE.read_release_version | ||
| # Read the current release version from the .xcconfig file and parse it into an AppVersion object |
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.
| version_short ||= VERSION_FILE.read_release_version | |
| # Read the current release version from the .xcconfig file and parse it into an AppVersion object | |
| # Read the current release version from the .xcconfig file and parse it into an AppVersion object, | |
| # unless already given | |
| version_short ||= VERSION_FILE.read_release_version |
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.
Alternatively, we could just remove the comment given the code is quite straightforward to read.
| version_short ||= VERSION_FILE.read_release_version | |
| # Read the current release version from the .xcconfig file and parse it into an AppVersion object | |
| version_short ||= VERSION_FILE.read_release_version |
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.
Updated on 0ba464b.
8a4865e to
ff9b7e2
Compare
…of just showing a warning
…hroughout code freeze
0ba464b to
9336d5f
Compare


See AINFRA-1478
Description
This PR enhances the version handling added in the previous iteration to assume the version received by the code freeze lane as the source of truth, ignoring potential mismatches between the ReleasesV2 version and the project's computed version.
Testing
You can make sure the calculated release branch, current version and the next version are correctly shown in the "Continue" prompt, remembering to answer
Nwhen asked to continue to cancel the code freeze.To avoid unexpected results, comment out the lines doing
ensure_git_status_cleanandFastlane::Helper::GitHelper.checkout_and_pull(DEFAULT_BRANCH)at the beginning of the lane.You can run the
start_code_freezelane, with and without the version parameters:bundle exec fastlane start_code_freezebundle exec fastlane start_code_freeze version:30.6This will be fully tested in the next release cycle during code freeze.