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

Canonicalize another exception name #12640

Merged
merged 1 commit into from
Aug 15, 2018
Merged

Conversation

1ec5
Copy link
Contributor

@1ec5 1ec5 commented Aug 15, 2018

Replaced one more exception name string with a formal symbol as a followup to #12583.

/cc @friedbunny

@1ec5 1ec5 added refactor macOS Mapbox Maps SDK for macOS labels Aug 15, 2018
@1ec5 1ec5 added this to the macos-v0.10.0 milestone Aug 15, 2018
@1ec5 1ec5 self-assigned this Aug 15, 2018
@1ec5 1ec5 requested a review from friedbunny August 15, 2018 15:48
@1ec5 1ec5 force-pushed the 1ec5-exception-standardization-12583 branch from d2a016e to 96ffe2d Compare August 15, 2018 15:52
@1ec5 1ec5 force-pushed the 1ec5-exception-standardization-12583 branch from 96ffe2d to 890b09d Compare August 15, 2018 15:53
Copy link
Contributor

@friedbunny friedbunny left a comment

Choose a reason for hiding this comment

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

🖥

@@ -8,6 +8,7 @@ Mapbox welcomes participation and contributions from everyone. Please read [CONT
* Fixed inconsistencies in exception naming. ([#12583](https://github.com/mapbox/mapbox-gl-native/issues/12583))
* Don't default-show text/icons that depend on the placement of a paired icon/text [#12483](https://github.com/mapbox/mapbox-gl-native/issues/12483)
* Fix the behavior of `-[MGLMapView visibleFeaturesAtPoint:]` near tile boundaries at high zoom. ([#12472](https://github.com/mapbox/mapbox-gl-native/issues/12472))
* Fixed inconsistencies in exception naming. ([#12583](https://github.com/mapbox/mapbox-gl-native/issues/12583))
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a dupe of L8.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I meant to make that change to the macOS changelog, not the iOS one. I’ve already fixed it.

@1ec5 1ec5 merged commit 2424582 into master Aug 15, 2018
@1ec5 1ec5 deleted the 1ec5-exception-standardization-12583 branch August 15, 2018 16:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
macOS Mapbox Maps SDK for macOS refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants