Skip to content
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.

Podspec / submodule updates for 5.6.0-beta.1, adapt to gl-native immutable GeoJSON sources #100

Closed
wants to merge 7 commits into from

Conversation

captainbarbosa
Copy link
Contributor

@captainbarbosa captainbarbosa commented Dec 12, 2019

This PR

@captainbarbosa captainbarbosa requested a review from a team December 12, 2019 20:24
@captainbarbosa captainbarbosa self-assigned this Dec 12, 2019
@captainbarbosa captainbarbosa requested review from jmkiley and fabian-guerra and removed request for a team December 12, 2019 20:26
Copy link
Contributor

@julianrex julianrex left a comment

Choose a reason for hiding this comment

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

Let's update INSTALL.md at the same time - unless that's in another PR?

@captainbarbosa captainbarbosa changed the title Podspec updates for 5.6.0-beta.1 Podspec / submodule updates for 5.6.0-beta.1 Dec 12, 2019
@captainbarbosa
Copy link
Contributor Author

This is on ⏸ until mapbox/mapbox-gl-native#16053 is resolved.

@captainbarbosa
Copy link
Contributor Author

captainbarbosa commented Dec 13, 2019

Possible fix going on in #101

@captainbarbosa captainbarbosa changed the title Podspec / submodule updates for 5.6.0-beta.1 Podspec / submodule updates for 5.6.0-beta.1, adapt to gl-native immutable GeoJSON sources Dec 13, 2019
Copy link
Contributor

@fabian-guerra fabian-guerra left a comment

Choose a reason for hiding this comment

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

Thank you for doing this. LGTM just two minor comments.

platform/ios/INSTALL.md Outdated Show resolved Hide resolved
platform/ios/INSTALL.md Outdated Show resolved Hide resolved
@captainbarbosa
Copy link
Contributor Author

Closing in favor of #103

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants