Skip to content

Conversation

@sileht
Copy link
Member

@sileht sileht commented Sep 3, 2025

Mainly libs change this hook, a common practice is to run the existing
hook after or before the custom code.

This avoids breaking monkeypatching of other libs.

@mergify
Copy link
Contributor

mergify bot commented Sep 3, 2025

🧪 CI Insights

Here's what we observed from your CI run for 9246029.

🟢 All jobs passed!

But CI Insights is watching 👀

jd
jd previously requested changes Sep 3, 2025
Copy link
Member

@jd jd left a comment

Choose a reason for hiding this comment

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

Care to add a test?

@sileht sileht force-pushed the devs/sileht/hook-fix/Ibe9bb54be35290b3cc0dda89ce713096afdb7e03 branch from f235c7f to 270a85b Compare September 3, 2025 16:50
@sileht sileht requested a review from jd September 3, 2025 16:51
@mergify mergify bot dismissed jd’s stale review September 3, 2025 16:51

Pull request has been modified.

Greesb
Greesb previously approved these changes Sep 4, 2025
@sileht sileht force-pushed the devs/sileht/hook-fix/Ibe9bb54be35290b3cc0dda89ce713096afdb7e03 branch from 270a85b to 2b2c466 Compare September 4, 2025 06:52
@mergify mergify bot dismissed Greesb’s stale review September 4, 2025 06:53

Pull request has been modified.

Mainly libs change this hook, a common practice is to run the existing
hook after or before the custom code.

This avoids breaking monkeypatching of other libs.

Change-Id: Ibe9bb54be35290b3cc0dda89ce713096afdb7e03
@sileht sileht force-pushed the devs/sileht/hook-fix/Ibe9bb54be35290b3cc0dda89ce713096afdb7e03 branch from 2b2c466 to 9246029 Compare September 4, 2025 06:56
@mergify mergify bot merged commit 3c9ccb7 into main Sep 4, 2025
9 checks passed
@mergify mergify bot deleted the devs/sileht/hook-fix/Ibe9bb54be35290b3cc0dda89ce713096afdb7e03 branch September 4, 2025 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants