Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support nested GeometryCollections #5

Open
jsiedentop opened this issue Feb 29, 2024 · 0 comments
Open

Support nested GeometryCollections #5

jsiedentop opened this issue Feb 29, 2024 · 0 comments

Comments

@jsiedentop
Copy link
Contributor

as described here

To maximize interoperability, implementations SHOULD avoid nested
GeometryCollections. Furthermore, GeometryCollections composed of a
single part or a number of parts of a single type SHOULD be avoided
when that single part or a single object of multipart type
(MultiPoint, MultiLineString, or MultiPolygon) could be used instead.

Related to the RFC, nested GeometryCollections are supported. I think this is something we should support in the future. To meet the SHOULD AVOID, this could be a linter warning and we provide a quick fix for this case. This strategy could allow the user to decide for themselves how they want to handle this case.

In case of turf, we would be strict and handle the warning as an error (as it is already handled in the current implementation).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant