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.

Sign up for GitHub

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

fix(instrumentation): do not import 'path' in browser runtimes #4386

Merged
merged 6 commits into from
Jan 3, 2024

Merge branch 'main' into fix/4373

b5e33c3
Select commit
Loading
Failed to load commit list.
Merged

fix(instrumentation): do not import 'path' in browser runtimes #4386

Merge branch 'main' into fix/4373
b5e33c3
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jan 3, 2024 in 0s

92.24% (+0.00%) compared to 0206181

View this Pull Request on Codecov

92.24% (+0.00%) compared to 0206181

Details

Codecov Report

Merging #4386 (b5e33c3) into main (0206181) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #4386   +/-   ##
=======================================
  Coverage   92.24%   92.24%           
=======================================
  Files         333      334    +1     
  Lines        9459     9463    +4     
  Branches     2009     2009           
=======================================
+ Hits         8725     8729    +4     
  Misses        734      734
Files Coverage Δ
...strumentation/src/instrumentationNodeModuleFile.ts 33.33% <100.00%> (ø)
...rumentation/src/platform/browser/noop-normalize.ts 100.00% <100.00%> (ø)
View more details on Codecov