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

Backmerge Commcare 2.53.1 #2685

Merged
merged 8 commits into from
Aug 1, 2023
Merged

Backmerge Commcare 2.53.1 #2685

merged 8 commits into from
Aug 1, 2023

Conversation

avazirna
Copy link
Contributor

@avazirna avazirna commented Jul 31, 2023

Summary

This PR back-merges changes from CommCare 2.53.1 into the master branch.

@avazirna avazirna marked this pull request as ready for review July 31, 2023 20:26
@@ -3,7 +3,7 @@
xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:tools="http://schemas.android.com/tools"
android:versionCode="106"
android:versionName="2.53">
android:versionName="2.53.1">
Copy link
Member

@ctsims ctsims Jul 31, 2023

Choose a reason for hiding this comment

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

I think the versionName tags shouldn't change on backmerges, right, or am I wrong on that? I think we try and reserve point released version ID's for non-main branches, and keep the main branch on a generic working version ID

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Correct, let me fix this

Copy link
Member

@ctsims ctsims left a comment

Choose a reason for hiding this comment

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

one question / feedback on general merge approach

@avazirna
Copy link
Contributor Author

@damagatchi retest this please

@avazirna
Copy link
Contributor Author

@damagatchi retest this please

@avazirna
Copy link
Contributor Author

@damagatchi retest this please

@avazirna avazirna merged commit c5f2eaf into master Aug 1, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants