Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Fix SIGINT signal at marked --help #3482

Closed
wants to merge 1 commit into from
Closed

Fix SIGINT signal at marked --help #3482

wants to merge 1 commit into from

Conversation

NOMADE55
Copy link
Contributor

@NOMADE55 NOMADE55 commented Oct 10, 2024 •
edited
Loading

Marked version:

v14.1.2

Markdown flavor: n/a

Description

Doing a Ctrl + C (^C ) caused an error on the child process spawned by marked.
The fix was to forward the signal and kill it in the spawned process.

Contributor

  • Test(s) exist to ensure functionality and minimize regression (if no tests added, list tests covering this PR); or,
  • no tests required for this PR.
  • If submitting new feature, it has been documented in the appropriate places.

Committer

In most cases, this should be a different person than the contributor.

@vercel Vercel
Copy link

vercel bot commented Oct 10, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
marked-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 10, 2024 7:41pm

@NOMADE55 NOMADE55 marked this pull request as ready for review October 10, 2024 19:44
@NOMADE55 NOMADE55 closed this by deleting the head repository Oct 10, 2024
@NOMADE55 NOMADE55 mentioned this pull request Oct 10, 2024
Merged
5 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Marked CLI] ^C causes manpage error
1 participant
@NOMADE55