Skip to content
This repository has been archived by the owner on Nov 14, 2019. It is now read-only.

Better dev exit #98

Merged
merged 3 commits into from
Jul 8, 2019
Merged

Better dev exit #98

merged 3 commits into from
Jul 8, 2019

Conversation

NicolasMahe
Copy link
Member

Dependant on #97

This PR implements a better exit system for the command dev.
Instead of killing the process, it destroys the streams.

@NicolasMahe NicolasMahe requested review from antho1404 and krhubert July 5, 2019 10:27
@NicolasMahe NicolasMahe force-pushed the feature/better-dev-exit branch from 426612d to 2f3c9df Compare July 5, 2019 10:32
Copy link
Member

@antho1404 antho1404 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not a big fan of the warnings in the logs but I don't have better for now

@antho1404 antho1404 merged commit d932bcf into master Jul 8, 2019
@antho1404 antho1404 deleted the feature/better-dev-exit branch July 8, 2019 04:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants