Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Update Bridges Subtree#2917

Merged
tomusdrw merged 7 commits intomasterfrom
hc-fix-bridges-subtree
Apr 23, 2021
Merged

Update Bridges Subtree#2917
tomusdrw merged 7 commits intomasterfrom
hc-fix-bridges-subtree

Conversation

@HCastano
Copy link
Copy Markdown
Contributor

This PR is a follow up to #2515 where the parity-bridges-common repo was added a a Git
Subtree.

Unknown to us at the time, squash merging that PR ended up putting us in a situation
where we were unable up update the bridges subtree via git subtree pull. This is
because we were already working with a squashed version of the bridge history (since we
added the repo through git subtree add ... --squash).

Note that this time we should not squash merge this PR so that in the future we can
easily update the bridges subtree using git subtree pull

@HCastano HCastano added the A0-please_review Pull request needs code review. label Apr 21, 2021
@HCastano HCastano requested a review from tomusdrw April 21, 2021 16:30
@HCastano HCastano added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Apr 21, 2021
Copy link
Copy Markdown
Contributor

@tomusdrw tomusdrw left a comment

Choose a reason for hiding this comment

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

LGTM! To anyone hitting the green button at the end: please make sure to NOT squash-merge this, but rather use regular merge as indicated in the description.

@ordian
Copy link
Copy Markdown

ordian commented Apr 22, 2021

how are we supposed to merge it?
image

@HCastano
Copy link
Copy Markdown
Contributor Author

how are we supposed to merge it?
image

So a squash-merge won't work for the above stated reasons, a rebase-merge won't work due to conflicts, and a regular merge is disabled.

So we either need someone with admin access to (temporarily) disable regular merges, or merge manually and push to master

@tomusdrw tomusdrw merged commit f4c0385 into master Apr 23, 2021
@tomusdrw tomusdrw deleted the hc-fix-bridges-subtree branch April 23, 2021 18:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants