Skip to content
This repository was archived by the owner on Feb 2, 2023. It is now read-only.

[Documentation] Improve readability of NS_OPTIONS in Swift - #2936

Merged
Adlai-Holler merged 1 commit into
masterfrom
HTSwiftReadability
Jan 27, 2017
Merged

[Documentation] Improve readability of NS_OPTIONS in Swift#2936
Adlai-Holler merged 1 commit into
masterfrom
HTSwiftReadability

Conversation

@hannahmbanana

Copy link
Copy Markdown
Contributor

No description provided.

@maicki

maicki commented Jan 26, 2017

Copy link
Copy Markdown
Contributor

@hannahmbanana If we remove the comments directly from the enums will we loose the Xcode documentation in ObjC if you option-click on it within a source code?

@maicki

maicki commented Jan 26, 2017

Copy link
Copy Markdown
Contributor

@hannahmbanana Can you please also rebase your branch on top of master. It seems like you merged in master a couple of times instead of rebasing.

@hannahmbanana

Copy link
Copy Markdown
Contributor Author

@maicki - interesting point about losing the option-click documentation. Shall we abandon this idea then?

@Adlai-Holler

Copy link
Copy Markdown
Contributor

I think we should only have the None option specified in the main NS_OPTIONS documentation, and only if the behavior of none isn't obvious.

@Adlai-Holler

Copy link
Copy Markdown
Contributor

Nice improvement!

@Adlai-Holler
Adlai-Holler merged commit 7d68ce3 into master Jan 27, 2017
@Adlai-Holler
Adlai-Holler deleted the HTSwiftReadability branch January 27, 2017 00:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants