New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Leverage built-in fetch API (remove axios) #5

Open

dlackty wants to merge 2 commits into detectlanguage:master from dlackty:remove-axios

dlackty commented Mar 24, 2024

This pull request removes the axios dependency in favor of the built-in fetch API.

Reduced bundle size: Eliminates the need for an external library.
Improved compatibility: Enables usage in environments like Cloudflare Workers and Vercel Functions, which often have limited library support.
Modernization: Aligns with the growing adoption of fetch API in Node.js. Node.js 16 reaching end-of-life, and the later versions since Node.js 18 have fetch API built-in.

dlackty added 2 commits

March 25, 2024 00:13


          Use fetch to replace axios

04465e2


          Update CI node versions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet