-
Notifications
You must be signed in to change notification settings - Fork 61
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
Dispose of watchers in serverless function code #3139
Dispose of watchers in serverless function code #3139
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #3139 +/- ##
=======================================
Coverage 32.97% 32.97%
=======================================
Files 75 75
Lines 5244 5265 +21
Branches 996 1000 +4
=======================================
+ Hits 1729 1736 +7
- Misses 3515 3529 +14
☔ View full report in Codecov by Sentry. |
|
a59923a
to
fe7b231
Compare
This change looks good to me... but it's not enough... When I have running the development vscode instance, plus started debugging the extension (with this PR included), then started Dev for a component in OS View and then...
Which brings me to WatchUtil.watchFileForContextChange(...) creating a File Watcher and at least two client views which invoke this method (and close the File Watchers) plus the test itself which never close the produced File Watchers. UPD: |
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.
Looks good to me
One part of redhat-developer#3120 Signed-off-by: David Thompson <davthomp@redhat.com>
fe7b231
to
8a49a36
Compare
Didn't realize you already had a PR open to do this. |
One part of #3120
Signed-off-by: David Thompson davthomp@redhat.com