fix: export named `fetch` from `node-fetch` polyfill

danielroe commented 1 year ago

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This adds a named fetch export for compatibility with node-fetch-native: https://github.com/unjs/node-fetch-native/blob/main/src/index.ts#L12.

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

pi0 commented 1 year ago

Seems a nice addition btw we are not aliasing node-fetch-native itself (src). Wondering what was the original issue?

rvmourik commented 1 year ago

@danielroe do you know when there will be a tagged release with this commit? We encounter this error and it stops us from deploying our newly migrated Nuxt3 website to Cloudflare workers.

Thanks in advance 🙏

danielroe commented 1 year ago

You should now be able to benefit from this from v1.0.1 ❤️

rvmourik commented 1 year ago

You should now be able to benefit from this from v1.0.1 ❤️

Thanks, works perfectly.

unjs / unenv