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

Implement Area based on traits #1115

Draft
wants to merge 4 commits into
base: traits
Choose a base branch
from

Conversation

kylebarron
Copy link
Member

  • I agree to follow the project's code of conduct.
  • I added an entry to CHANGES.md if knowledge of this change could be valuable to users.

An exploration of #1113, based on top of #1114. Having a bunch of traits is verbose 😕 . Is there any way to group lots of traits as one? Wishing for something like

pub mod AreaTraits {
    pub use super::AreaPoint;
    pub use super::AreaLineString;
}

where importing AreaTraits would add all of the internal traits to scope. Guessing that doesn't exist, and probably for a good reason.

@frewsxcv frewsxcv marked this pull request as draft January 19, 2024 01:35
@frewsxcv
Copy link
Member

I hope it's okay I converted this to a draft PR, since it's not really ready for a review until we figure out a plan for geo-traits

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