Skip to content

Commit

Permalink
removing resolveTargetbrowserMapped wrapper method, seems it's no lon…
Browse files Browse the repository at this point in the history
…ger needed
  • Loading branch information
Jarred de Beer authored and Jarred de Beer committed Apr 23, 2021
1 parent 95b2b8e commit fd37bda
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 34 deletions.
20 changes: 1 addition & 19 deletions packages/node-resolve/src/package/resolvePackageTarget.js
Expand Up @@ -10,24 +10,6 @@ function includesInvalidSegments(pathSegments, moduleDirs) {
.some((t) => ['.', '..', ...moduleDirs].includes(t));
}

async function resolvePackageTargetBrowserMapped(context, { target, subpath, pattern, internal }) {
const resolvedTarget = await resolvePackageTarget(context, {
target,
subpath,
pattern,
internal
});
if (
resolvedTarget &&
resolvedTarget.startsWith('./') &&
context.packageBrowserField &&
context.packageBrowserField[resolvedTarget]
) {
return context.packageBrowserField[resolvedTarget];
}
return resolvedTarget;
}

async function resolvePackageTarget(context, { target, subpath, pattern, internal }) {
if (typeof target === 'string') {
if (!pattern && subpath.length > 0 && !target.endsWith('/')) {
Expand Down Expand Up @@ -129,4 +111,4 @@ async function resolvePackageTarget(context, { target, subpath, pattern, interna
throw new InvalidPackageTargetError(context, `Invalid exports field.`);
}

export default resolvePackageTargetBrowserMapped;
export default resolvePackageTarget;

This file was deleted.

This file was deleted.

This file was deleted.

0 comments on commit fd37bda

Please sign in to comment.