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

Renaming cluster modules #846

Merged
merged 2 commits into from
Jul 16, 2017
Merged

Renaming cluster modules #846

merged 2 commits into from
Jul 16, 2017

Conversation

DenisCarriere
Copy link
Member

@DenisCarriere DenisCarriere commented Jul 16, 2017

$ yarn add @turf/clusters
yarn add v0.24.5
info No lockfile found.
[1/4] 🔍  Resolving packages...
warning @turf/clusters@4.5.2: Module has been renamed to '@turf/cluster-kmeans'
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
[4/4] 📃  Building fresh packages...

@DenisCarriere DenisCarriere added this to the 4.6.0 milestone Jul 16, 2017
@DenisCarriere DenisCarriere self-assigned this Jul 16, 2017
@DenisCarriere DenisCarriere merged commit 560a3c1 into master Jul 16, 2017
@DenisCarriere DenisCarriere deleted the renaming-cluster-modules branch July 16, 2017 19:03
@stebogit stebogit mentioned this pull request Aug 3, 2017
@stebogit
Copy link
Collaborator

stebogit commented Aug 3, 2017

@DenisCarriere how did you add the warning message to the yarn/npm installation? I didn't find any code related to that in Turf. Is it an npm thing?

@DenisCarriere
Copy link
Member Author

It's an npm thing, you can find out how to add warning message here: https://docs.npmjs.com/cli/deprecate

@stebogit
Copy link
Collaborator

stebogit commented Aug 3, 2017

Thanks, nice to know!

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