search

dimapio / map-gl

Interactive maps in the browser
MIT License
0 stars 0 forks source link

Confirm whether the `@types/geojson` dependency is required #11

Closed nsiebenaller closed 3 days ago

nsiebenaller commented 6 days ago

mapbox-gl uses @types/geojson in conjunction with it's own types. Currently, we're adding @types/geojson as a dependency, but I don't want to have to micromanage matching our version of the dependency with mapbox-gl.

It might be possible to remove this dependency and maintain correct types.

https://github.com/dimapio/map-gl/blob/d74331eea3dd69d70743452bed31899e28a46705/package.json#L20

nsiebenaller commented 3 days ago

mapbox-gl uses @types/geojson as a dependency so we get it automatically. I'll move @types/geojson to a peer dependency to indicate that it should be provided.