Skip to content

module: ensure successful dynamic import returns the same result #46662

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

Merged
nodejs-github-bot merged 13 commits into from
Jul 26, 2023
+255 −27
Merged

module: ensure successful dynamic import returns the same result #46662

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
0f1fe00
module: ensure successful import returns the same result
aduh95 Apr 14, 2023
3bf7e02
create `ModuleResolveMap` class
aduh95 Jun 10, 2023
9f36bf7
Apply suggestions from code reviews
aduh95 Jul 2, 2023
aabe393
Make the resolve cache a resolve cache
aduh95 Jul 21, 2023
be69edd
revert changes in `test/es-module/test-esm-initialization.mjs`
aduh95 Jul 21, 2023
dd06864
fix missing object
aduh95 Jul 22, 2023
631d7c6
add benchmark
aduh95 Jul 22, 2023
c279d18
add `data:` URL to benchmark
aduh95 Jul 22, 2023
56a8a07
DRY
aduh95 Jul 24, 2023
abf105a
Apply suggestions from code review
aduh95 Jul 25, 2023
00ea14c
fix lint
aduh95 Jul 25, 2023
a791dc9
Update lib/internal/modules/esm/module_map.js
aduh95 Jul 26, 2023
3638858
fix lint
aduh95 Jul 26, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fix lint
  • Loading branch information
@aduh95
aduh95 committed Jul 26, 2023
commit 3638858a03b86898381c042f8dcf4815e0682e9b
2 changes: 1 addition & 1 deletion lib/internal/modules/esm/module_map.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ class ResolveCache extends SafeMap {
// so that different import statements with the same assertions are always treated
// as identical.
const keys = ObjectKeys(importAssertions);

if (keys.length === 0) {
return specifier + '::';
}
Expand Down