Skip to content

Support pull hooks#21486

Merged
niik merged 3 commits intodevelopmentfrom
pull-hooks
Jan 30, 2026
Merged

Support pull hooks#21486
niik merged 3 commits intodevelopmentfrom
pull-hooks

Conversation

@niik
Copy link
Member

@niik niik commented Jan 21, 2026

Description

Based on #21461

Screenshots

Release notes

Notes:

@niik niik changed the title Add support for hooks related to pull Support pull hooks Jan 21, 2026
Base automatically changed from push-hooks to development January 21, 2026 10:23
@niik niik mentioned this pull request Jan 22, 2026
@niik niik mentioned this pull request Jan 22, 2026
Comment on lines +56 to +57
'prepare-commit-msg',
'commit-msg',
Copy link
Member

Choose a reason for hiding this comment

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

These two are duplicated and can be deleted, right?

Suggested change
'prepare-commit-msg',
'commit-msg',

...gitRebaseArguments(),
'pull',
...(await getDefaultPullDivergentBranchArguments(repository)),
...(enableRecurseSubmodulesFlag() ? ['--recurse-submodules'] : []),
Copy link
Member

Choose a reason for hiding this comment

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

This feature flag got removed in development with #21188

Suggested change
...(enableRecurseSubmodulesFlag() ? ['--recurse-submodules'] : []),
'--recurse-submodules',

Copy link
Member

@sergiou87 sergiou87 left a comment

Choose a reason for hiding this comment

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

Code looks good and I did some testing, it seems to work as expected! 💪

:shipit:

@niik niik merged commit 6e01551 into development Jan 30, 2026
7 checks passed
@niik niik deleted the pull-hooks branch January 30, 2026 10:49
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