Skip to content

Conversation

@smallsaucepan
Copy link
Member

Minimally intrusive conversion of turf-meta to typescript. Rather than try to untangle internal type inconsistencies, leave the public interface and runtime as is and sprinkle a liberal dose of @ts-expect-error statements. Once decided on a resolution strategy we can remove those as we go.

Note the excludeWrapCoord parameter on coordReduce wasn't defined in the d.ts. It is documented and a parameter at runtime, so have added it as an optional fourth param. Believe this would be non-breaking.

Please provide the following when creating a PR:

  • Meaningful title, including the name of the package being modified.
  • Summary of the changes.
  • Heads up if this is a breaking change.
  • Any issues this resolves.
  • Inclusion of your details in the contributors field of package.json - you've earned it! 👏
  • Confirmation you've read the steps for preparing a pull request.

…n try to untangle internal type inconsistencies, leave the public interface and runtime as is and sprinkle a liberal dose of @ts-expect-error statements. Once decided on a resolution strategy we can remove those as we go.
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.

1 participant