-
Notifications
You must be signed in to change notification settings - Fork 18
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
Implemented issue #81 (allow stopping dapr apps) #179
Conversation
@@ -1,3 +1,6 @@ | |||
/* eslint-disable @typescript-eslint/no-unsafe-call */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: we prefer to disable linting rules for individual lines rather than whole files. (And there are often tricks to make the TS compiler happy, than can avoid having to disable certain kinds of rules at all.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of comments, and then I think it can go in after your related daprd
path changes are merged.
Implemented the Stop App command to address #81 and added documentation to the readme. There is a bug with the extension not being able to pick up dapr processes (#169) so testing this will require a temporary fix similar to whats proposed here (#170)
@philliphoff @pkedy @artursouza