Skip to content

Read version from properties file in increment-version workflow#1516

Open
DavDo wants to merge 1 commit into
developmentfrom
MOB-5436-read-version-from-properties
Open

Read version from properties file in increment-version workflow#1516
DavDo wants to merge 1 commit into
developmentfrom
MOB-5436-read-version-from-properties

Conversation

@DavDo

@DavDo DavDo commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Jira issue:
https://glia.atlassian.net/browse/MOB-5436

What was solved?

The "Increment project version" Bitrise workflow (_increment_project_version) was failing: the Bitrise Gradle Mirrors init script (bitrise-build-cache CLI) pollutes stdout of every Gradle invocation, so ./gradlew -q printCurrentVersionName returned the mirror log lines along with the version. This produced an invalid branch name in the Create PR step and failed the build with fatal: 'project-version-increment/[Bitrise' is not a valid branch name.

Now the version is read directly from version.properties — the same file the workflow updates and commits — making the step immune to any Gradle stdout noise. Same fix as salemove/android-sdk#1073.

Additional info:

  • Is the feature sufficiently tested? All tests fixed? Necessary unit, acceptance, snapshots added? Check that at least new public classes & methods are covered with unit tests — CI-only change, no SDK code touched; extraction verified locally against version.properties
  • Did you add logging beneficial for troubleshooting of customer issues? — N/A, CI workflow change
  • Did you add new logging? We would like the logging between platforms to be similar. Refer to Logging from Android SDKsThings to consider for newly added logs in Confluence for more information. — N/A

Screenshots:

N/A

The Bitrise Gradle Mirrors init script pollutes stdout of every Gradle
invocation, so `./gradlew -q printCurrentVersionName` returned the mirror
log lines along with the version, producing an invalid branch name in the
Create PR step. Read widgets.versionName directly from version.properties
instead, making the step immune to Gradle stdout noise.
@DavDo DavDo requested a review from a team as a code owner July 3, 2026 14:14
@snyk-io

snyk-io Bot commented Jul 3, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

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.

1 participant