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

Deprecate style class API #7785

Merged
merged 2 commits into from
Jan 26, 2017

Conversation

fabian-guerra
Copy link
Contributor

According to #7577 and #2875 we should mark style methods and properties as deprecated because we will not support them on future releases.

The Android SDK has already deprecated same API.

This is a temporarily PR to branch release-ios-v3.4.0, this should be included in release-ios-v3.4.1

@fabian-guerra fabian-guerra added iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS runtime styling labels Jan 19, 2017
@fabian-guerra fabian-guerra requested a review from boundsj January 19, 2017 15:13
@fabian-guerra fabian-guerra self-assigned this Jan 19, 2017
@fabian-guerra fabian-guerra added this to the ios-v3.4.1 milestone Jan 19, 2017
@boundsj boundsj added the ⚠️ DO NOT MERGE Work in progress, proof of concept, or on hold label Jan 19, 2017
@boundsj
Copy link
Contributor

boundsj commented Jan 19, 2017

Looks 👍 to me. Added do not merge until we have a release branch to put this.

@1ec5 1ec5 changed the title Deprecate style api methods Deprecate style class API Jan 20, 2017
@1ec5
Copy link
Contributor

1ec5 commented Jan 20, 2017

To clarify, this is about the style class API, not the style API (aka the runtime styling API).

@boundsj boundsj modified the milestones: ios-v3.5.0, ios-v3.4.1 Jan 24, 2017
@boundsj boundsj removed the ⚠️ DO NOT MERGE Work in progress, proof of concept, or on hold label Jan 24, 2017
@boundsj
Copy link
Contributor

boundsj commented Jan 24, 2017

@fabian-guerra I changed the milestone to be 3.5.0. I think you can go ahead and target this against the master branch and merge it.

@1ec5 1ec5 modified the milestones: ios-v3.4.1, ios-v3.5.0 Jan 24, 2017
Copy link
Contributor

@1ec5 1ec5 left a comment

Choose a reason for hiding this comment

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

Can you add an entry to both the iOS and macOS changelogs to note that the style class API has been deprecated? Once you do that, please merge.

@fabian-guerra fabian-guerra force-pushed the fabian-deprecate-style-api-methods branch 3 times, most recently from 9af2844 to 140abfc Compare January 25, 2017 22:08
@friedbunny friedbunny force-pushed the fabian-deprecate-style-api-methods branch from 140abfc to 917f3a5 Compare January 26, 2017 03:18
@friedbunny friedbunny merged commit 8ac9975 into release-ios-v3.4.0 Jan 26, 2017
@friedbunny friedbunny deleted the fabian-deprecate-style-api-methods branch January 26, 2017 03:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS runtime styling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants