Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Console] #47809 remove exit() call in last SignalHandler #17544

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.

Sign up for GitHub

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

Jump to bottom
Closed
fabpot opened this issue Dec 9, 2022 · 2 comments
Closed

[Console] #47809 remove exit() call in last SignalHandler #17544

fabpot opened this issue Dec 9, 2022 · 2 comments
Labels
Console hasPR A Pull Request has already been submitted for this issue.
Milestone
6.3

Comments

@fabpot
Copy link
Member

fabpot commented Dec 9, 2022

Q A
Feature PR symfony/symfony#48299
PR author(s) @akuzia
Merged in 6.3
@fabpot fabpot added this to the 6.3 milestone Dec 9, 2022
@javiereguiluz
Copy link
Member

@nicolas-grekas do you think this change is something we should mention in the docs or is it an internal detail? Thanks.

@alexandre-daubois
Copy link
Member

From what I understood of the PR and context, I think we can add a little something to the documentation. I'm giving it a try, I'll let you tell me if what I add is relevant 😄

@alexandre-daubois alexandre-daubois mentioned this issue Jan 24, 2023
Merged
javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this issue Jan 25, 2023
@javiereguiluz
minor symfony#17805 [Console] Remove exit() call in last SignalHandle…
011181f 
…r (alexandre-daubois)

This PR was merged into the 6.3 branch.

Discussion
----------

[Console] Remove exit() call in last SignalHandler

Fixes symfony#17544

Commits
-------

aeb0ccc [Console] Remove exit() call in last SignalHandler
@xabbuh xabbuh closed this as completed Jan 29, 2023
@xabbuh xabbuh added the hasPR A Pull Request has already been submitted for this issue. label Jan 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Console hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Milestone
6.3
Development

No branches or pull requests
5 participants
@fabpot @javiereguiluz @xabbuh @alexandre-daubois @carsonbot