Uh oh!
There was an error while loading. Please reload this page.
feat(hooks): support interactive hook scripts - #1925
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@## master #1925 +/- ##
=======================================
Coverage 98.23% 98.23% =======================================
Files 61 61 Lines 2772 2779 +7 =======================================
+ Hits 2723 2730 +7
Misses 49 49 ☔ View full report in Codecov by Sentry. |
98d622f to
8746afcCompareUh oh!
There was an error while loading. Please reload this page.
bearomorphism
commented
Apr 12, 2026
Please fix the coverage pipeline, thanks |
PhilipNelson5
commented
Apr 12, 2026
@bearomorphism I added tests for |
da45b62 to
d483e68Comparebearomorphism
commented
Apr 15, 2026
Thank you for fixing the pipeline failures! @PhilipNelson5 @woile please review this feature |
Uh oh!
There was an error while loading. Please reload this page.
5109dfa to
53cc1aeCompare53cc1ae to
47f3372ComparePhilipNelson5
commented
May 6, 2026
@bearomorphism I've brought in the changes you made in #1941 and mirrored them in this feature. |
bearomorphism
commented
May 7, 2026
Thanks! will review when I have bandwidth |
bearomorphism
left a comment
There was a problem hiding this comment.
LGTM. I see some duplicated test code but it's non-blocking.
Wait for @woile to do the final review
Uh oh!
There was an error while loading. Please reload this page.
woile
commented
May 8, 2026
thanks 🎉 🚀 |
Description
When a pre/post bump hook needs user input it fails because stdin, stdout, and stderr are redirected. This feature adds a new cmd utility to run a command without capturing/redirecting input or output to the subprocess. This allows interactive hooks to request input and display output to the user interactively.
Checklist
Was generative AI tooling used to co-author this PR?
Code Changes
uv run poe alllocally to ensure this change passes linter check and testsUpdate the documentation for the changesDocumentation Changes
Runuv run poe doclocally to ensure the documentation pages renders correctlyCheck and fix any broken links (internal or external)Expected Behavior
When a pre/post bump hook tries to get user input, it successfully does so.
Steps to Test This Pull Request
cz bump.Additional Context