Skip to content
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

[#316] Use build number from CI/CD #451

Closed
wants to merge 7 commits into from

Conversation

edgarss
Copy link
Contributor

@edgarss edgarss commented Mar 8, 2023

What happened 👀

CI/CD tools like Bitrise, GitHubAction, and Codemagic provide the build number so we can use that for the build number of the application. We will update our templates for Bitrise and GitHubAction to get the following benefit:

It is easier to track the commit of a version.
Make the build number more predictable.
Avoid duplicate build numbers when rebuilding the same commit (after registering a new device, we must re-deploy the app).
CI machines don't have to clone the whole project history to get the correct number of commits, reducing the overall build time and the cost of running CI/CD.

Insight 📝

  • added ios-bump-version step in github workflow
  • added set-xcode-build-number step in bitrise

Proof Of Work 📹

coming soon

@vnntsu
Copy link
Contributor

vnntsu commented Mar 9, 2023

@edgarss FYI, you can create the PR as a draft during development, and you then can set it ready for review. So no need to mark [WIP] on the title.

@edgarss edgarss marked this pull request as draft March 10, 2023 02:53
@vnntsu vnntsu modified the milestones: 4.3.0, 4.4.0 Mar 31, 2023
@ducbm051291 ducbm051291 linked an issue Apr 19, 2023 that may be closed by this pull request
@ducbm051291 ducbm051291 force-pushed the feature/316-build-number-from-ci branch from 06af457 to 393c0da Compare April 19, 2023 04:21
@ducbm051291 ducbm051291 self-assigned this Apr 19, 2023
@ducbm051291 ducbm051291 force-pushed the feature/316-build-number-from-ci branch from 06261d3 to f3d5e0f Compare April 19, 2023 06:54
@ducbm051291 ducbm051291 changed the base branch from develop to feature/330-fix-signing April 19, 2023 06:54
@ducbm051291 ducbm051291 changed the title [WIP][316] Use build number from CI/CD [WIP][#316] Use build number from CI/CD Apr 19, 2023
@ducbm051291 ducbm051291 force-pushed the feature/316-build-number-from-ci branch from 57598e6 to d45f2a4 Compare April 21, 2023 06:29
@ducbm051291 ducbm051291 changed the title [WIP][#316] Use build number from CI/CD [#316] Use build number from CI/CD Apr 25, 2023
Base automatically changed from feature/330-fix-signing to develop April 28, 2023 03:15
@vnntsu vnntsu modified the milestones: 4.4.0, 4.5.0 May 8, 2023
@vnntsu vnntsu removed this from the 4.5.0 milestone Jun 5, 2023
@vnntsu vnntsu added this to the 4.6.0 milestone Jun 5, 2023
@suho
Copy link
Member

suho commented Jul 21, 2023

@ducbm051291 This pull request opened months ago, please continue the work

@blyscuit blyscuit modified the milestones: 4.6.0, 4.8.0 Aug 9, 2023
@blyscuit blyscuit removed this from the 4.8.0 milestone Sep 7, 2023
@suho suho requested review from suho and removed request for suho September 21, 2023 07:12
@ducbm051291 ducbm051291 force-pushed the feature/316-build-number-from-ci branch from 2c00b54 to d3c55a5 Compare October 23, 2023 02:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI/CD - Use build number provided by CI/CD services
6 participants