Skip to content

Make types/rbush a dependency#184

Merged
bdon merged 1 commit intoprotomaps:mainfrom
pjonsson:move-rbush-types
Mar 10, 2025
Merged

Make types/rbush a dependency#184
bdon merged 1 commit intoprotomaps:mainfrom
pjonsson:move-rbush-types

Conversation

@pjonsson
Copy link
Copy Markdown
Contributor

@pjonsson pjonsson commented Mar 7, 2025

The types exposed by protomaps-leaflet
import rbush, which causes an error
for projects that have no direct
dependency on types/rbush:

node_modules/protomaps-leaflet/dist/esm/index.d.ts:3:19 - error TS7016: Could not find a declaration file for module 'rbush'. '/files/node_modules/rbush/rbush.js' implicitly has an 'any' type.

Make types/rbush a dependency instead,
similar to how commit ca5ba0a did
it for types/css-font-loading-module.

The types exposed by protomaps-leaflet
import rbush, which causes an error
for projects that have no direct
dependency on types/rbush:

node_modules/protomaps-leaflet/dist/esm/index.d.ts:3:19 - error TS7016: Could not find a declaration file for module 'rbush'. '/files/node_modules/rbush/rbush.js' implicitly has an 'any' type.

Make types/rbush a dependency instead,
similar to how commit ca5ba0a did
it for types/css-font-loading-module.
@bdon bdon merged commit 0589556 into protomaps:main Mar 10, 2025
1 check passed
@bdon
Copy link
Copy Markdown
Member

bdon commented Mar 10, 2025

Thanks!

@bdon
Copy link
Copy Markdown
Member

bdon commented Mar 10, 2025

published in v4.1.1

@pjonsson pjonsson deleted the move-rbush-types branch March 11, 2025 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants