Skip to content

Swift style guide Chinese edition #47

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

Merged
merged 13 commits into from
Jul 7, 2014
Merged

Swift style guide Chinese edition #47

merged 13 commits into from
Jul 7, 2014

Conversation

leonerd42
Copy link
Contributor

Please modify it if anything is not appropriate.

leonerd42 added 11 commits July 4, 2014 06:58
Signed-off-by: mrahmiao <mrahmiao@gmail.com>
Signed-off-by: mrahmiao <mrahmiao@gmail.com>
Signed-off-by: mrahmiao <mrahmiao@gmail.com>
Signed-off-by: mrahmiao <mrahmiao@gmail.com>
Signed-off-by: mrahmiao <mrahmiao@gmail.com>
Signed-off-by: mrahmiao <mrahmiao@gmail.com>
Signed-off-by: mrahmiao <mrahmiao@gmail.com>
Signed-off-by: mrahmiao <mrahmiao@gmail.com>
Signed-off-by: mrahmiao <mrahmiao@gmail.com>
Signed-off-by: mrahmiao <mrahmiao@gmail.com>
Signed-off-by: mrahmiao <mrahmiao@gmail.com>
@jackwu95
Copy link
Contributor

jackwu95 commented Jul 6, 2014

Awesome! I took the liberty to read over this and its quite polished.
Quick comments:

  • You missed the semicolons section in the table of contents
  • The last part in the 命名 (naming) section should benefit from some translation.

More eyes on this would be helpful.

@leonerd42
Copy link
Contributor Author

You mean the last few lines of code?

leonerd42 added 2 commits July 6, 2014 17:31
…the code part in naming section

Signed-off-by: mrahmiao <mrahmiao@gmail.com>
Signed-off-by: mrahmiao <mrahmiao@gmail.com>
@jackwu95
Copy link
Contributor

jackwu95 commented Jul 6, 2014

Yes that is exactly what I meant, looks great!

rwenderlich added a commit that referenced this pull request Jul 7, 2014
Swift style guide Chinese edition
@rwenderlich rwenderlich merged commit f99044c into kodecocodes:master Jul 7, 2014
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.

3 participants