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

Improve release merge script for deleted files #4353

Closed

Conversation

ernestognw
Copy link
Member

Description

Recently, the release scripts failed trying to merge the release-v4.9 branch back to master. The reason is that merging with -X theirs strategy doesn't solve conflicts for files modified in release-v4.9 and deleted in master, which is the case for multiple contracts we've removed for 5.0

With this change, the conflicts will be solved by removing the files as they're removed in master.

@changeset-bot
Copy link

changeset-bot bot commented Jun 14, 2023

⚠️ No Changeset found

Latest commit: cfcfb83

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ernestognw ernestognw requested review from frangio and Amxx June 14, 2023 17:06
@ernestognw ernestognw added ignore-changeset on hold Put on hold for some reason that must be specified in a comment. labels Jun 14, 2023
@ernestognw
Copy link
Member Author

Put on hold since we may want to remove that part of the workflow. We've realized there are a few edge cases that require manual review and are frequent enough to prefer manual intervention over a fine-tuned automation.

@frangio
Copy link
Contributor

frangio commented Jun 14, 2023

Yes, I vote for removing automatic merge resolution. We should create the merge/release-vX.Y branch pointing to the latest commit of release-vX.Y and create the PR, but expect manual resolution from that point on. It may not be necessary in some cases and GitHub might just work.

@frangio
Copy link
Contributor

frangio commented Jun 16, 2023

Superceded by #4362.

@frangio frangio closed this Jun 16, 2023
@frangio frangio removed the on hold Put on hold for some reason that must be specified in a comment. label Jun 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants