Skip to content
This repository was archived by the owner on Dec 25, 2024. It is now read-only.

fix: wrong basename import #72

Merged
merged 1 commit into from
Nov 22, 2021
Merged

fix: wrong basename import #72

merged 1 commit into from
Nov 22, 2021

Conversation

upupming
Copy link
Contributor

@upupming upupming commented Nov 21, 2021

Sorry, I used an old version of node, I think there is nothing wrong with the current LTS node.

@upupming upupming closed this Nov 21, 2021
@upupming upupming deleted the fix-import branch November 21, 2021 15:17
@antfu
Copy link
Member

antfu commented Nov 22, 2021

Why closing? I think it's a valid fix

@upupming
Copy link
Contributor Author

upupming commented Nov 22, 2021

'path/posix' is supported in Node.js v15.3.0, and the current LTS node is v16.13.0, so I think we could use 'path/posix' here~ You could merge this anyway

@upupming upupming restored the fix-import branch November 22, 2021 02:09
@antfu antfu reopened this Nov 22, 2021
@antfu
Copy link
Member

antfu commented Nov 22, 2021

This is more likely introduced by IDE autocompletion, this is not special enough to force users to be on node v16.

@antfu antfu merged commit 595d749 into unplugin:main Nov 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants