Migrate @turf/buffer to TypeScript, ignoring errors#2994
Migrate @turf/buffer to TypeScript, ignoring errors#2994
Conversation
smallsaucepan
left a comment
There was a problem hiding this comment.
Thanks @mfedderly. One more hurdle down.
|
Hey, this is a fantastic effort. I know the plan is to move away from jsts, but any chance that in the meantime #2937 could get rolled into this update? I'd be happy to re-do the PR in whatever typescript file comes out of your work 🙂 |
|
@plymer I'm trying to focus on #2991 instead of this, and I'm hesitating to expose these configurations until we know we're settled on a library that does provide this. But I am intending to add this eventually. Unfortunately I'm less convinced that clipper2-ts is a library we want to depend on going forward, so I might go and port clipper2 from scratch myself before we can land that. |
|
Cheers, I appreciate the update and the roadmap ❤️ |
|
Is your lack of desire to use clipper2 as-is an ergonomics/efficiency thing with all the transformation? |
|
Sadly no, I have concerns about accuracy past |
|
Converting to draft as it's likely we'll favour #2991 instead |
I'd like to do this more properly, but that will require a major version release.
In the meantime, we can do a low-effort job that gets us migrated away from Javascript-first packages and consolidate on TypeScript instead to simplify some of the build tooling.