Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: browser field resolving relative to path to itself #34

Merged
merged 1 commit into from
Dec 17, 2023

Conversation

Boshen
Copy link
Member

@Boshen Boshen commented Dec 17, 2023

fixes #32

Copy link

codecov bot commented Dec 17, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (42c89e3) 94.12% compared to head (9e2ab5b) 94.13%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #34      +/-   ##
==========================================
+ Coverage   94.12%   94.13%   +0.01%     
==========================================
  Files          11       11              
  Lines        2263     2268       +5     
==========================================
+ Hits         2130     2135       +5     
  Misses        133      133              

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

@Boshen Boshen enabled auto-merge (squash) December 17, 2023 12:40
Copy link

codspeed-hq bot commented Dec 17, 2023

CodSpeed Performance Report

Merging #34 will not alter performance

Comparing browser-field-resolve-self (9e2ab5b) with main (b3d8f7c)

Summary

✅ 2 untouched benchmarks

@Boshen Boshen merged commit c4432a5 into main Dec 17, 2023
19 checks passed
@Boshen Boshen deleted the browser-field-resolve-self branch December 17, 2023 12:40
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

should allow "./e.js": "./e.js" in browser field
1 participant