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

Replace manual try/finally with using declarations in TS parser #10

Draft
wants to merge 6 commits into
base: parsing-pipeline-using
Choose a base branch
from

Conversation

nicolo-ribaudo
Copy link
Owner

@nicolo-ribaudo nicolo-ribaudo commented Jan 15, 2024

Better viewed with whitespace diff disabled, as there are a lot of indentation changes: https://github.com/nicolo-ribaudo/babel/pull/10/files?w=1

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (3c17c6c) 93.07% compared to head (daf390e) 93.20%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@                    Coverage Diff                     @@
##           parsing-pipeline-using      #10      +/-   ##
==========================================================
+ Coverage                   93.07%   93.20%   +0.13%     
==========================================================
  Files                         523      523              
  Lines                       95010    94940      -70     
  Branches                    13083    13067      -16     
==========================================================
+ Hits                        88429    88492      +63     
+ Misses                       6406     6300     -106     
+ Partials                      175      148      -27     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants