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

Turf v2.0.0 #741

Merged
merged 1 commit into from
Oct 6, 2021
Merged

Turf v2.0.0 #741

merged 1 commit into from
Oct 6, 2021

Conversation

1ec5
Copy link
Contributor

@1ec5 1ec5 commented Oct 6, 2021

Updated SPM and CocoaPods manifests for MapboxMaps to require Turf v2.x. Turf v2.0.0 comes with a number of backwards-incompatible changes compared to either v1.x or v2.0.0-rc.1 related to GeoJSON types, as seen in #715. However, there have been no backwards-incompatible changes since v2.0.0-rc.2, so this PR is solely about updating the dependency files.

/cc @mapbox/maps-ios

@1ec5 1ec5 added the dependencies Pull requests that update a dependency file label Oct 6, 2021
@1ec5 1ec5 added this to the ga milestone Oct 6, 2021
@1ec5 1ec5 requested a review from macdrevx October 6, 2021 02:29
@1ec5 1ec5 self-assigned this Oct 6, 2021
@1ec5 1ec5 force-pushed the 1ec5-turf-v2.0.0 branch 2 times, most recently from 2271538 to 3b25b59 Compare October 6, 2021 02:34
@@ -24,6 +24,6 @@ Pod::Spec.new do |m|
m.dependency 'MapboxCoreMaps', '10.0.0'
m.dependency 'MapboxCommon', '20.0.0'
m.dependency 'MapboxMobileEvents', '1.0.5'
m.dependency 'Turf', '2.0.0-rc.2'
m.dependency 'Turf', '~> 2.0'
Copy link
Contributor Author

@1ec5 1ec5 Oct 6, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The patch version is omitted for consistency with SPM’s Package.Dependency.package(name:url:from:), which allows compatible minor versions.

@macdrevx

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants