Skip to content

[brockit] Adding drop! command for B🚀#37057

Merged
cdesouza-chromium merged 1 commit into
masterfrom
brockit+drop-command
Jun 6, 2026
Merged

[brockit] Adding drop! command for B🚀#37057
cdesouza-chromium merged 1 commit into
masterfrom
brockit+drop-command

Conversation

@cdesouza-chromium
Copy link
Copy Markdown
Collaborator

This command is a sibling to reassign!, in the sense that it also
creates a commit with a subcommand tag to guide a rebase event. In this
case though, the drop! indicates that a target commit is supposed to
be dropped during rebase.

Similar to ressign! handling, this type of pin can only be processed
with --squash-minor-bumps, and the target commit cannot be another
pinned commit.

This change is being implemented for rebase v2 only.

Bug: brave/brave-browser#56150

@cdesouza-chromium cdesouza-chromium requested a review from a team as a code owner June 6, 2026 19:49
@cdesouza-chromium cdesouza-chromium added the CI/skip Do not run CI builds (except noplatform) label Jun 6, 2026
@cdesouza-chromium cdesouza-chromium self-assigned this Jun 6, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 6, 2026

📋 Code Owners Summary

4 file(s) changed, 4 with assigned owners

1 team(s) affected: @brave/chromium-rebase-team


Owners and Their Files

@brave/chromium-rebase-team — 4 file(s)

This command is a sibling to `reassign!`, in the sense that it also
creates a commit with a subcommand tag to guide a rebase event. In this
case though, the `drop!` indicates that a target commit is supposed to
be dropped during rebase.

Similar to `ressign!` handling, this type of pin can only be processed
with `--squash-minor-bumps`, and the target commit cannot be another
pinned commit.

This change is being implemented for rebase v2 only.

Bug: brave/brave-browser#56150
@brave-builds
Copy link
Copy Markdown
Collaborator

Warning

You have got a presubmit warning. Please address it if possible.

Found 1 lines longer than 80 characters (first 5 shown).

Items:

tools/cr/brockit.py, line 3100, 86 chars

@cdesouza-chromium cdesouza-chromium merged commit a7e89eb into master Jun 6, 2026
19 checks passed
@cdesouza-chromium cdesouza-chromium deleted the brockit+drop-command branch June 6, 2026 23:45
@brave-builds brave-builds added this to the 1.93.x - Nightly milestone Jun 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/skip Do not run CI builds (except noplatform)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants