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

Merge 19.4.1 hotfix into 19.5 release branch #18188

Merged
merged 4 commits into from
Mar 23, 2022

Conversation

mokagio
Copy link
Contributor

@mokagio mokagio commented Mar 23, 2022

As expected when merging the branch for the release currently in code freeze into an hotfix branch with the purpose of opening a PR to track the hotfix process into the latest release branch, we got conflicts in config/Version.internal.xcconfig and config/Version.public.xcconfig because both hotfix 19.4.1 and release 19.5 change the version compared to their common ancestor.

I resolved the conflict, as usual, by keeping the value from 19.5 which is the latest. To verify this, notice that there is not diff for either Version.*.xcconfig.

In fact, the diff is totally empty. This is because the hotfix comes from a PR that originally targeted 19.4, but was not merged in time. The value of this PR is merely to track the fact that an hotfix occurred in the Git history of our main branch.

mokagio and others added 4 commits March 23, 2022 10:35
As expected when merging the branch for the release currently in code
freeze into an hotfix branch with the purpose of opening a PR to track
the hotfix process into the latest release branch, we got conflicts in
`config/Version.internal.xcconfig` and `config/Version.public.xcconfig`
because both hotfix 19.4.1 and release 19.5 change the version compared
to their common ancestor.

I resolved the conflict, as usual, by keeping the value from 19.5 which
is the latest.
@peril-wordpress-mobile
Copy link

Messages
📖 This PR has the 'Releases' label: some checks will be skipped.

Generated by 🚫 dangerJS

@mokagio mokagio marked this pull request as ready for review March 23, 2022 03:53
@wpmobilebot
Copy link
Contributor

You can test the Jetpack changes on this Pull Request by downloading it from AppCenter here with build number: pr18188-100d54e. IPA is available here. If you need access to this, you can ask a maintainer to add you.

@wpmobilebot
Copy link
Contributor

You can test the WordPress changes on this Pull Request by downloading it from AppCenter here with build number: pr18188-100d54e. IPA is available here. If you need access to this, you can ask a maintainer to add you.

@mokagio mokagio requested a review from a team March 23, 2022 04:47
@mokagio mokagio enabled auto-merge March 23, 2022 04:47
Copy link
Contributor

@AliSoftware AliSoftware left a comment

Choose a reason for hiding this comment

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

Empty diff makes sense given the fix from hotfix was a cherry-pick from one applied in release branch

@mokagio mokagio merged commit 183cf15 into release/19.5 Mar 23, 2022
@mokagio mokagio deleted the merge/release-19.4.1-into-19.5 branch March 23, 2022 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants