search

axiomhq / axiom-js

Official language bindings and library extensions for Axiom
https://axiom.co
MIT License
97 stars 14 forks source link

chore: remove cross-fetch #77

Closed ImLunaHey closed 1 year ago

ImLunaHey commented 1 year ago

All modern browsers have fetch and for the backend nodejs LTS has fetch. This removed 300kb+ from the package.

For the minority that still need this they can import the polyfill before this package for the same effect.

ImLunaHey commented 1 year ago

A quick fix for the failing browser test would be to add this back as a devDependancy and import it at the top of the test file.

ImLunaHey commented 1 year ago

@schehata curious what's holding up the CI. 🤔

Is there a reason the integration tests aren't completing?