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

Adds properties parameter to @turf/centroid and @turf/center #769

Merged
merged 6 commits into from
May 29, 2017

Conversation

stebogit
Copy link
Collaborator

@DenisCarriere in order to add the parameter to center-of-mass we need to merge this to master first, as center-of-mass depends on centroid.

Ref: #755

@DenisCarriere
Copy link
Member

DenisCarriere commented May 29, 2017

Interestingly you can actually update all 3 at once and center-of-mass will use the latest version of centroid since lerna uses symbolic links to all the dependencies of each modules.

$ lerna bootstrap

- Add properties support to center-of-mass
- Update Readmes
- Update typescript definitions
- Update tests using glob
- Update test fixtures
- Update benchmark results
- Added switch statement CenterOfMass
@DenisCarriere
Copy link
Member

@stebogit Updated significantly the tests:

  • Add properties support to center-of-mass
  • Update Readmes
  • Update typescript definitions
  • Update tests using glob
  • Update test fixtures
  • Update benchmark results
  • Added switch statement CenterOfMass
  • Added Typescript tests (types.ts)

@DenisCarriere DenisCarriere merged commit ff47ae2 into master May 29, 2017
@DenisCarriere DenisCarriere deleted the centroid-properties branch May 29, 2017 18:20
@DenisCarriere DenisCarriere added this to the 4.4.0 milestone May 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants