Skip to content

Conversation

glyh
Copy link
Member

@glyh glyh commented Oct 3, 2025

This is porting merge-clean script in #17906 to compatible.

The script is completely rewritten, previously we're checking on:

  • BUILD_BRANCH -> COMPATIBILITY_BRANCH

And now we're checking on the below instead:

  • (BUILD_BRANCH -> PR_BASE_BRANCH) -> COMPATIBILITY_BRANCH

@glyh glyh requested a review from a team as a code owner October 3, 2025 15:42
@glyh glyh changed the title Fix merges-cleanly.sh to actually check on 3-way merge Rewrite merges-cleanly.sh to actually check on 3-way merge Oct 3, 2025
@glyh glyh changed the title Rewrite merges-cleanly.sh to actually check on 3-way merge Rewrite merges-cleanly.sh to relax the condition. Oct 13, 2025
@glyh glyh force-pushed the lyh/new-check-merge-cleanly-script branch from 65508f9 to 93c6e0f Compare October 13, 2025 02:36
echo "Checking conflicts: (${PR_BRANCH} -> ${BASE_BRANCH}) -> ${SYNC_BRANCH}"


# Adapted from this stackoverflow answer: https://stackoverflow.com/a/10856937
Copy link
Member Author

Choose a reason for hiding this comment

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

This comment is not telling what's being done at all.

@glyh glyh force-pushed the lyh/new-check-merge-cleanly-script branch from 93c6e0f to 11e8c8a Compare October 13, 2025 02:38
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.

1 participant