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

feat: resolveModuleExportNames and findExportNames #63

Merged
merged 3 commits into from Aug 3, 2022
Merged

Conversation

pi0
Copy link
Member

@pi0 pi0 commented Aug 3, 2022

Expose fineExportNames as a shortcut for findExports that only returns export names.

Add new resolveModuleExportNames that resolves and reads a module and lists it's available exports using static analayzes.

Note: I didn't include resolveModuleExports with additional loc data since seems not useful. But name uses Names suffix so we have space to introduce it later.

@pi0 pi0 changed the title feat: resolveModuleExportNames and fineExportNames feat: resolveModuleExportNames and findExportNames Aug 3, 2022
@codecov
Copy link

codecov bot commented Aug 3, 2022

Codecov Report

Merging #63 (27140aa) into main (7039f54) will increase coverage by 5.16%.
The diff coverage is 53.84%.

❗ Current head 27140aa differs from pull request most recent head 4c7ddcf. Consider uploading reports for the commit 4c7ddcf to get more accurate results

@@            Coverage Diff             @@
##             main      #63      +/-   ##
==========================================
+ Coverage   52.89%   58.05%   +5.16%     
==========================================
  Files          13       13              
  Lines        2159     2172      +13     
  Branches      179      218      +39     
==========================================
+ Hits         1142     1261     +119     
+ Misses        847      701     -146     
- Partials      170      210      +40     
Impacted Files Coverage Δ
src/analyze.ts 78.83% <53.84%> (-1.85%) ⬇️
lib/import-meta-resolve/errors.js 75.00% <0.00%> (+0.28%) ⬆️
src/resolve.ts 62.90% <0.00%> (+1.61%) ⬆️
src/utils.ts 50.00% <0.00%> (+3.22%) ⬆️
lib/import-meta-resolve/resolve.js 49.15% <0.00%> (+10.05%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@pi0 pi0 merged commit a699573 into main Aug 3, 2022
@pi0 pi0 deleted the feat/resolve-exports branch August 3, 2022 10:19
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

1 participant