Skip to content
This repository was archived by the owner on Apr 3, 2024. It is now read-only.

chore: remove dead code for supporting node < 10#1141

Merged
mctavish merged 2 commits intogoogleapis:mainfrom
mctavish:remove-legacy
May 25, 2023
Merged

chore: remove dead code for supporting node < 10#1141
mctavish merged 2 commits intogoogleapis:mainfrom
mctavish:remove-legacy

Conversation

@mctavish
Copy link
Copy Markdown
Contributor

This is a cleanup change that removes code that hasn't been used since before Node 10. No functional changes expected as at the moment we only support Node >= 12.

@mctavish mctavish requested review from a team May 25, 2023 16:29
@product-auto-label product-auto-label bot added size: l Pull request size is large. api: clouddebugger Issues related to the googleapis/cloud-debug-nodejs API. labels May 25, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented May 25, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +7.08 🎉

Comparison is base (81ae406) 65.68% compared to head (acfdf02) 72.77%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1141      +/-   ##
==========================================
+ Coverage   65.68%   72.77%   +7.08%     
==========================================
  Files          21       20       -1     
  Lines        1798     1605     -193     
  Branches      355      314      -41     
==========================================
- Hits         1181     1168      -13     
+ Misses        530      346     -184     
- Partials       87       91       +4     
Impacted Files Coverage Δ
src/agent/v8/debugapi.ts 100.00% <100.00%> (+17.64%) ⬆️

... and 3 files with indirect coverage changes

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

@mctavish mctavish requested a review from jasonborg May 25, 2023 17:01
@mctavish mctavish added kokoro:run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels May 25, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label May 25, 2023
@yoshi-kokoro yoshi-kokoro removed the kokoro:run Add this label to force Kokoro to re-run the tests. label May 25, 2023
@mctavish mctavish added kokoro:run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels May 25, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label May 25, 2023
@mctavish mctavish added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 25, 2023
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels May 25, 2023
@mctavish mctavish merged commit 47a6fc8 into googleapis:main May 25, 2023
@mctavish mctavish deleted the remove-legacy branch May 25, 2023 17:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: clouddebugger Issues related to the googleapis/cloud-debug-nodejs API. size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants