Skip to content

[deps] update resolve#248

Open
ljharb wants to merge 1 commit intomainfrom
ljharb-patch-1
Open

[deps] update resolve#248
ljharb wants to merge 1 commit intomainfrom
ljharb-patch-1

Conversation

@ljharb
Copy link
Copy Markdown
Member

@ljharb ljharb commented Mar 5, 2026

This updates to resolve@next, which for this package's uses is a nonbreaking update.

What this allows in the future, however, is exports-aware resolution.

This updates to resolve@next, which for this package's uses is a nonbreaking update.

What this allows in the future, however, is `exports`-aware resolution.
@ljharb ljharb requested a review from nicolo-ribaudo March 5, 2026 19:50
@JLHwung
Copy link
Copy Markdown
Collaborator

JLHwung commented Apr 30, 2026

@ljharb Could you rebase and run yarn dedupe?

@ljharb
Copy link
Copy Markdown
Member Author

ljharb commented Apr 30, 2026

hm, this package no longer seems to include resolve. What does it use for resolution now?

@JLHwung
Copy link
Copy Markdown
Collaborator

JLHwung commented Apr 30, 2026

hm, this package no longer seems to include resolve. What does it use for resolution now?

Previously resolve serves as polyfill for Node.js < 8.9 which does not support the paths option. Now that the library drops Node.js < 20 support, it now only uses the native require.resolve.

@ljharb
Copy link
Copy Markdown
Member Author

ljharb commented Apr 30, 2026

do you need to resolve anything based on a range of user-supported node versions? (even if the tool requires node >= 20)

@JLHwung
Copy link
Copy Markdown
Collaborator

JLHwung commented Apr 30, 2026

do you need to resolve anything based on a range of user-supported node versions? (even if the tool requires node >= 20)

I don't recall we have such an API to resolve something in a way that Node.js 6 would resolve.

@nicolo-ribaudo Do you think there is a use case for that?

@ljharb
Copy link
Copy Markdown
Member Author

ljharb commented Apr 30, 2026

If so, you'll want resolve@next, if not, this PR should be closed :-)

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.

2 participants