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

Improves cleanup process when reloading emulated functions in debug mode. #5878

Merged
merged 5 commits into from
May 23, 2023

Conversation

taeold
Copy link
Contributor

@taeold taeold commented May 22, 2023

Instead of spawning a new process that exposes the debugger endpoint, the Functions Emulator will attempt to explicitly kill the existing process before loading the function with the updated source code.

Possible fix for #5508, #5653

@taeold taeold changed the title Improve cleanup process when reloading emulated functions in debug mode. Improves cleanup process when reloading emulated functions in debug mode. May 22, 2023
@codecov-commenter
Copy link

codecov-commenter commented May 22, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.01 ⚠️

Comparison is base (4c5d8c9) 54.97% compared to head (33c0d8e) 54.97%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5878      +/-   ##
==========================================
- Coverage   54.97%   54.97%   -0.01%     
==========================================
  Files         333      333              
  Lines       22980    22982       +2     
  Branches     4711     4712       +1     
==========================================
  Hits        12634    12634              
- Misses       9218     9220       +2     
  Partials     1128     1128              
Impacted Files Coverage Δ
src/emulator/functionsEmulator.ts 7.70% <0.00%> (-0.03%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@taeold taeold enabled auto-merge (squash) May 23, 2023 18:11
@taeold taeold merged commit 7909fc6 into master May 23, 2023
31 of 32 checks passed
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.

None yet

3 participants