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

adjust version code logic from v24 on #205

Merged
merged 2 commits into from
Nov 8, 2023
Merged

Conversation

gabrielittner
Copy link
Member

From version 24.1.0 on (next year) the version code will move the part for the minor version one digit to the right. This leaves an unused 0 between the minor version and the major version and is purely for readability purposes.

Examples:

  • 24.1.0: 24010000 -> 24001000
  • 24.10.0: 24100000 -> 24010000
  • 24.24.0: 24240000 -> 24024000
  • 24.44.0: 24440000 -> 24044000

For regular release builds the last 2 digits are used for patches (both beta patches and hotfix) and the 100 digit is unused.

The logic for untagged builds which we use for nightly builds is adjusted to use the last 3 digits. The first of those 3 is still used for the day of week. Since that leaves only 2 digits and 99 commits per release/week is not enough we now use the time of day there. This allows to still do multiple "nightly" builds per day for testing purposes. The only limitation is that you can only do one every 15 minutes but that is fine for one-off tests.

@gabrielittner gabrielittner changed the title Version code 24 adjust version code logic from v24 on Nov 7, 2023
Copy link
Collaborator

@jalalawqati jalalawqati left a comment

Choose a reason for hiding this comment

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

Nice 👍

@gabrielittner gabrielittner merged commit 6219704 into main Nov 8, 2023
3 checks passed
@gabrielittner gabrielittner deleted the version-code-24 branch November 8, 2023 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants