Renamed pointer-geometry dependency to @mapbox/pointer-geometry by eko · Pull Request #437 · google-map-react/google-map-react

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@eko

Copy link Copy Markdown

@eko eko commented

Aug 21, 2017

Hello,

This dependency has been renamed so when running npm install, we get:

$ npm install

npm WARN deprecated point-geometry@0.0.0: This module has moved: please install @mapbox/point-geometry instead

Thank you

Horat1us, rmacqueen, ken92, and alexgsummer reacted with thumbs up emoji

@Horat1us

Copy link Copy Markdown

Horat1us commented

Sep 4, 2017

+1 Why not to merge this PR?

@rmacqueen

Copy link Copy Markdown

Agreed, please merge. Warnings are quite annoying and is easy fix

@istarkov

Copy link Copy Markdown

Collaborator

istarkov commented

Sep 9, 2017

has anyone checked releases log of pointer geometry, I have no time to check this

@Horat1us

Copy link Copy Markdown

Horat1us commented

Sep 9, 2017

They just add some methods:

https://github.com/mapbox/point-geometry/blob/master/HISTORY.md

@eko

Copy link Copy Markdown

Author

eko commented

Sep 9, 2017

+1, I don't see any BC break

@Exomnius

Copy link Copy Markdown

Contributor

Exomnius commented

Oct 9, 2017

Seems ok. Please merge this for next version release.

@itsmichaeldiego

Copy link Copy Markdown

Member

@eko Thanks for your PR, this changes have been handled in #462

@lock

Copy link Copy Markdown

lock bot commented

Dec 1, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators

Dec 1, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@eko @Horat1us @rmacqueen @istarkov @Exomnius @itsmichaeldiego