Skip to content

Conversation

roccomuso
Copy link
Contributor

Export wretch.

So we can replace wretch with @exodus/fetch

@roccomuso roccomuso requested a review from ChALkeR February 24, 2025 15:59
@roccomuso roccomuso self-assigned this Feb 24, 2025
@roccomuso roccomuso requested a review from feri42 February 25, 2025 15:05
Copy link
Contributor Author

@roccomuso roccomuso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@roccomuso roccomuso merged commit 575c66a into master Feb 26, 2025
1 check passed
@roccomuso roccomuso deleted the rocco/export-wretch branch February 26, 2025 09:15
Copy link

🎉 This PR is included in version 1.7.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ChALkeR
Copy link
Contributor

ChALkeR commented Mar 1, 2025

No, this adds wretch to bundles that don't need it.
Use @exodus/fetch/wretch import instead and revert this

It's exported here:

fetch/package.json

Lines 56 to 61 in cc523c2

"./wretch": {
"browser": "./wretch.browser.js",
"react-native": "./wretch.js",
"node": "./wretch.js",
"default": "./wretch.js"
},

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants