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 the SIGINT signal in signals.go to listen for the user enter ctrl+c to exit the application operation #3609

Closed
wants to merge 3 commits into from

Conversation

Suyghur
Copy link
Contributor

@Suyghur Suyghur commented Oct 9, 2023

add the SIGINT signal in signals.go to listen for the user enter ctrl+c to exit the application operation, and i think it's useful in coding.

…ter ctrl+c to exit the application operation
@codecov
Copy link

codecov bot commented Oct 9, 2023

Codecov Report

Merging #3609 (cbcb1a8) into master (ff230c4) will not change coverage.
The diff coverage is 50.00%.

❗ Current head cbcb1a8 differs from pull request most recent head 80fd7a9. Consider uploading reports for the commit 80fd7a9 to get more accurate results

Files Coverage Δ
core/proc/signals.go 43.33% <50.00%> (ø)

@Suyghur
Copy link
Contributor Author

Suyghur commented Oct 9, 2023

please help to code review @kevwan

@Suyghur Suyghur closed this Oct 9, 2023
@Suyghur Suyghur deleted the feat/signals branch October 9, 2023 12:27
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.

1 participant