Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add signal handler for process execution #1008

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

ivanduplenskikh
Copy link
Contributor

@ivanduplenskikh ivanduplenskikh commented Jan 8, 2024

Changes: Added signal handler for process execution to kill process with proper signal. This PR also adds unit tests.

@DergachevE DergachevE force-pushed the users/ivanduplenskikh/add-signal-handler-for-process branch from 01e0dd9 to c3ebdb9 Compare April 26, 2024 10:23
@ivanduplenskikh ivanduplenskikh marked this pull request as ready for review May 10, 2024 12:04
@ivanduplenskikh ivanduplenskikh requested review from a team as code owners May 10, 2024 12:04
@ivanduplenskikh ivanduplenskikh force-pushed the users/ivanduplenskikh/add-signal-handler-for-process branch 3 times, most recently from 1da2bdd to 8f8dc8f Compare June 3, 2024 11:48
@ivanduplenskikh ivanduplenskikh force-pushed the users/ivanduplenskikh/add-signal-handler-for-process branch from 8f8dc8f to 7c0de0d Compare June 3, 2024 11:48
@ivanduplenskikh ivanduplenskikh force-pushed the users/ivanduplenskikh/add-signal-handler-for-process branch from ef6fdf0 to 6a1d011 Compare June 3, 2024 14:32
node/toolrunner.ts Outdated Show resolved Hide resolved
ivanduplenskikh and others added 3 commits June 5, 2024 11:31
Co-authored-by: Konstantin Tyukalov <52399739+KonstantinTyukalov@users.noreply.github.com>
@ivanduplenskikh ivanduplenskikh force-pushed the users/ivanduplenskikh/add-signal-handler-for-process branch from 6ca3aca to a591821 Compare July 1, 2024 14:45
@ivanduplenskikh ivanduplenskikh force-pushed the users/ivanduplenskikh/add-signal-handler-for-process branch from b4c2801 to 7b8d407 Compare July 12, 2024 12:10
@max-zaytsev
Copy link
Contributor

/azp where

Copy link

Azure DevOps orgs getting events for this repository:

@max-zaytsev
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

None yet

3 participants