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

Flag MapRecorder with @_spi(Internal) #1582

Merged
merged 2 commits into from
Sep 16, 2022

Conversation

maios
Copy link
Contributor

@maios maios commented Sep 16, 2022

Flag @_spi(Internal) for MapRecorder so we can enable Release configuration in benchmark tests.

Pull request checklist:

  • Describe the changes in this PR, especially public API changes.

PRs must be submitted under the terms of our Contributor License Agreement CLA.

@maios maios requested a review from OdNairy September 16, 2022 12:11
@maios maios requested a review from a team as a code owner September 16, 2022 12:11
@maios maios merged commit f7bf202 into main Sep 16, 2022
@maios maios deleted the feature/make-map-recorder-spi-internal branch September 16, 2022 12:46
persidskiy pushed a commit that referenced this pull request Feb 24, 2023
* Fix for Style/updateLayer fail if nested properties dictionary contains nil (#1565)

- Error happens due to layer's properties contains nil values for nested keys, for these keys, default values will not be applied so they would appear as nil when updating a layer. The fix is to only apply nil encoding for the root properties of the dictionary.

* Flag MapRecorder with @_spi(Internal) (#1582)

* Remove unused images when updating point annotations (#1583)

* Parse GeoJSON data on a background queue (#1576)

* Fix toString conversion on iOS 12 (#1584)

* Use asynchronous GeoJSON marshaling in annotation managers (#1588)

* update generated docs (#1586)

* Fix GeoJSONData.url incorrectly converting to string (#1587)

* Expose distance-from-center and pitch expressions  (#1559)

* Bump MapboxCoreMaps and MapboxCommon (#1589)

* Expose opacity for location puck (#1585)

* Fix merge issues

Co-authored-by: Mai Mai <mai.mai@mapbox.com>
Co-authored-by: Roman Laitarenko <roman.laitarenko@mapbox.com>
Co-authored-by: Ankur Khandelwal <ankur.khandelwal@mapbox.com>
Co-authored-by: Tobrun <tobrun.van.nuland@gmail.com>
Co-authored-by: ZiZi <44972592+ZiZasaurus@users.noreply.github.com>
OdNairy pushed a commit that referenced this pull request Aug 22, 2023
* Import PR template from the public repo

* Update PULL_REQUEST_TEMPLATE.md
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