Skip to content

fix: handle SIGHUP in watcher#2764

Merged
vmaerten merged 1 commit intogo-task:mainfrom
mateenali66:fix/sighup-watcher
Apr 9, 2026
Merged

fix: handle SIGHUP in watcher#2764
vmaerten merged 1 commit intogo-task:mainfrom
mateenali66:fix/sighup-watcher

Conversation

@mateenali66
Copy link
Copy Markdown
Contributor

watcher currently restarts on SIGINT and SIGTERM but not SIGHUP. added SIGHUP to the signal handler in signals.go so it gets the same treatment — closes #2642

Signed-off-by: Mateen Anjum <mateenali66@gmail.com>
@FredrikAugust
Copy link
Copy Markdown

FredrikAugust commented Apr 7, 2026

This would be great to see land:) Conductor.build uses SIGHUP to stop the "run" task.

@vmaerten
Copy link
Copy Markdown
Member

vmaerten commented Apr 9, 2026

Thanks for your contribution!

@vmaerten vmaerten merged commit 44e1350 into go-task:main Apr 9, 2026
14 of 19 checks passed
vmaerten added a commit that referenced this pull request Apr 9, 2026
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.

Watcher interrupt on SIGHUP

3 participants