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: use resolver function to resolve module #535

Merged
merged 13 commits into from
Dec 18, 2024
Merged

feat: use resolver function to resolve module #535

merged 13 commits into from
Dec 18, 2024

Conversation

fi3ework
Copy link
Member

@fi3ework fi3ework commented Dec 8, 2024

Summary

Fix #238.
Fix #250.
Fix #553.
Fix #407.
Close #548.

As of now, we got getResolve function in external callback. Invoke getResolve once and cache the resolver to use afterward for performance concern.

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copy link

netlify bot commented Dec 8, 2024

Deploy Preview for rslib ready!

Name Link
🔨 Latest commit 2f03097
🔍 Latest deploy log https://app.netlify.com/sites/rslib/deploys/6762ce70cc3b74000802a74b
😎 Deploy Preview https://deploy-preview-535--rslib.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

codspeed-hq bot commented Dec 8, 2024

CodSpeed Performance Report

Merging #535 will not alter performance

Comparing main-files (2f03097) with main (72587ed)

Summary

✅ 5 untouched benchmarks

@Timeless0911
Copy link
Collaborator

#251
#407

Also should be considered in this PR.

@fi3ework fi3ework force-pushed the main-files branch 2 times, most recently from a69fe4d to 06ac05a Compare December 12, 2024 11:42
@fi3ework fi3ework force-pushed the main-files branch 3 times, most recently from 124441d to 53cb5ec Compare December 16, 2024 09:47
@fi3ework fi3ework marked this pull request as ready for review December 16, 2024 11:21
@fi3ework fi3ework requested review from Timeless0911 and Copilot and removed request for Timeless0911 December 16, 2024 11:21

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 35 out of 50 changed files in this pull request and generated no comments.

Files not reviewed (15)
  • packages/core/tests/snapshots/config.test.ts.snap: Language not supported
  • scripts/dictionary.txt: Language not supported
  • tests/integration/auto-extension/type-commonjs/config-override/package.json: Language not supported
  • tests/integration/auto-extension/type-commonjs/default/package.json: Language not supported
  • tests/integration/auto-extension/type-commonjs/false-bundleless/package.json: Language not supported
  • tests/integration/auto-extension/type-commonjs/false/package.json: Language not supported
  • tests/integration/auto-extension/type-module/config-override/package.json: Language not supported
  • tests/integration/auto-extension/type-module/default/package.json: Language not supported
  • tests/integration/auto-extension/fixtures/src/common.ts: Evaluated as low risk
  • packages/core/src/config.ts: Evaluated as low risk
  • packages/core/src/css/cssConfig.ts: Evaluated as low risk
  • packages/core/src/types/config.ts: Evaluated as low risk
  • packages/core/src/types/utils.ts: Evaluated as low risk
  • packages/core/src/utils/helper.ts: Evaluated as low risk
  • pnpm-lock.yaml: Evaluated as low risk
packages/core/src/config.ts Outdated Show resolved Hide resolved
packages/core/src/config.ts Outdated Show resolved Hide resolved
packages/core/src/config.ts Show resolved Hide resolved
packages/core/src/config.ts Show resolved Hide resolved
packages/core/src/types/config.ts Show resolved Hide resolved
website/docs/en/config/lib/redirect.mdx Outdated Show resolved Hide resolved
website/docs/en/config/lib/redirect.mdx Outdated Show resolved Hide resolved
@Timeless0911
Copy link
Collaborator

I think the pr title should be changed.

@fi3ework fi3ework changed the title fix: use resolver function to correctly resolve feat: use resolver function to resolve module Dec 18, 2024
Copy link
Collaborator

@Timeless0911 Timeless0911 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update docs and tests about externals in bundleless mode in next pr.

@Timeless0911 Timeless0911 merged commit acadf07 into main Dec 18, 2024
18 checks passed
@Timeless0911 Timeless0911 deleted the main-files branch December 18, 2024 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants