Skip to content

mercurial: add a config knob to apply patches to the tip revision - #105

Merged
marco-c merged 1 commit into
mozilla:masterfrom
cgsheeh:uplift-patch-apply-tip
Feb 20, 2024
Merged

marco-c merged 1 commit into
mozilla:masterfrom
cgsheeh:uplift-patch-apply-tip

Conversation

@cgsheeh

@cgsheeh cgsheeh commented Feb 15, 2024

Copy link
Copy Markdown
Member

Add a use_latest_revision config knob to the repo configuration that
causes patch stacks to be applied to the latest revision in the repo
(tip) instead of the parent revision of the base commit in the stack.
This is useful for uplift revisions, where the parent of the base
may not exist in the repo, and where applying the patch to the latest
is desirable to get early feedback on any merge conflicts that will be
encountered on landing.

Move the logic for determining the initial base_revision into a
separate function to enable a simple unit test.

@cgsheeh

cgsheeh commented Feb 15, 2024

Copy link
Copy Markdown
Member Author

I think the CI failure is unrelated here.

Closes mozilla/code-review#2057

@marco-c

marco-c commented Feb 20, 2024

Copy link
Copy Markdown
Collaborator

@cgsheeh I fixed the linting issue on master, could you rebase?

Add a `use_latest_revision` config knob to the repo configuration that
causes patch stacks to be applied to the latest revision in the repo
(`tip`) instead of the parent revision of the base commit in the stack.
This is useful for uplift revisions, where the parent of the base
may not exist in the repo, and where applying the patch to the latest
is desirable to get early feedback on any merge conflicts that will be
encountered on landing.

Move the logic for determining the initial `base_revision` into a
separate function to enable a simple unit test.
@cgsheeh
cgsheeh force-pushed the uplift-patch-apply-tip branch from 177024c to 96bb677 Compare February 20, 2024 15:38
@cgsheeh

cgsheeh commented Feb 20, 2024

Copy link
Copy Markdown
Member Author

@cgsheeh I fixed the linting issue on master, could you rebase?

@marco-c rebased. 👍

@marco-c marco-c left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks!

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