You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Sep 26, 2024. It is now read-only.
As a newcomer to this repository, I'm pretty confused with declarationdeclare module "*" in typescript extraneous declaration file.
It seems like a smelly code for me, combining with no monorepo tooling it caused an irritating bug of every module being linked to this declaration, effectively ruining all type checks.
Seems like this line was added to help out with absent type on datamaps module, but probably it was an overkill to add a wildcard for that.
Proposals
Replace wildcard module definition to datamaps module definition
Either update datamaps to 0.5.9 which has type definitions and write own react components or make a PR with types to react-datamaps package (propose your own solution!)
Be more strict on wildcards in general :)
The text was updated successfully, but these errors were encountered:
Given that the GeoMap Page is now dead, we may as well just remove datamaps for now.
Well, there were some historical reasons why I went stupid and merged the PR before we addressed all the code quality issues. We should definitely improve it.
@luixo If this breaks your productivity, I suggest you address this issue first.
Story
As a newcomer to this repository, I'm pretty confused with declaration
declare module "*"
in typescript extraneous declaration file.It seems like a smelly code for me, combining with no monorepo tooling it caused an irritating bug of every module being linked to this declaration, effectively ruining all type checks.
Seems like this line was added to help out with absent type on
datamaps
module, but probably it was an overkill to add a wildcard for that.Proposals
datamaps
module definitiondatamaps
to 0.5.9 which has type definitions and write own react components or make a PR with types toreact-datamaps
package (propose your own solution!)The text was updated successfully, but these errors were encountered: