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

fix: rework app exit to account for running bots #9532

Merged
merged 16 commits into from
Apr 17, 2023

Conversation

OEvgeny
Copy link
Collaborator

@OEvgeny OEvgeny commented Mar 7, 2023

Description

This reworks exit flow:

  • Added exit indication and fixed client exit cycle (by @sw-joelmut)
  • Reworked server exit functionality to account for platform specifics and enforce proper exit (by @OEvgeny)

Task Item

fixes #9059

Screenshot

@OEvgeny OEvgeny marked this pull request as ready for review March 14, 2023 17:32
@OEvgeny OEvgeny changed the title fix: close server listeners upon app exit fix: rework app exit to account for running bots Mar 14, 2023
sw-joelmut
sw-joelmut previously approved these changes Mar 14, 2023
@coveralls
Copy link

coveralls commented Mar 15, 2023

Coverage Status

Coverage: 54.551% (-0.008%) from 54.559% when pulling 7dfd03b on OEvgeny:fix/close-listeners into efc9138 on microsoft:main.

@cwhitten
Copy link
Member

@OEvgeny there are conflicts on this PR now

@OEvgeny
Copy link
Collaborator Author

OEvgeny commented Apr 17, 2023

@cwhitten updated, thanks!

@cwhitten cwhitten merged commit 39f01c2 into microsoft:main Apr 17, 2023
@OEvgeny OEvgeny deleted the fix/close-listeners branch April 18, 2023 02:26
@cwhitten cwhitten mentioned this pull request Aug 15, 2023
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.

Bot remains as a background process after closing Composer
4 participants