From 1ec3bd4b3d35bc2871c02a75782f9144c1c4477c Mon Sep 17 00:00:00 2001 From: Rob Rix Date: Mon, 25 Aug 2014 08:20:24 -0400 Subject: [PATCH 1/3] =?UTF-8?q?Don=E2=80=99t=20leave=20trailing=20whitespa?= =?UTF-8?q?ce=20section?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Brings in the no trailing whitespace section from https://github.com/github/objective-c-style-guide/pull/58. --- README.md | 3 +++ 1 file changed, 3 insertions(+) diff --git a/README.md b/README.md index c4018a9..147626a 100644 --- a/README.md +++ b/README.md @@ -18,6 +18,9 @@ then open a pull request. :zap: * Tabs, not spaces. * End files with a newline. * Make liberal use of vertical whitespace to divide code into logical chunks. + * Don’t leave trailing whitespace. + * Not even leading indentation on blank lines. + #### Prefer implicit getters on read-only properties and subscripts From 3599cabadda3fb184c6742d5f99496da6e3d13c8 Mon Sep 17 00:00:00 2001 From: Rob Rix Date: Mon, 25 Aug 2014 08:20:44 -0400 Subject: [PATCH 2/3] Update README.md --- README.md | 1 - 1 file changed, 1 deletion(-) diff --git a/README.md b/README.md index 147626a..5da1fd5 100644 --- a/README.md +++ b/README.md @@ -21,7 +21,6 @@ then open a pull request. :zap: * Don’t leave trailing whitespace. * Not even leading indentation on blank lines. - #### Prefer implicit getters on read-only properties and subscripts When possible, omit the `get` keyword on read-only computed properties and From e3028d2ee4ecca24fb2a8e1db3c84d6c76d3ec62 Mon Sep 17 00:00:00 2001 From: Rob Rix Date: Mon, 25 Aug 2014 08:21:15 -0400 Subject: [PATCH 3/3] Space out, man. --- README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.md b/README.md index 5da1fd5..99bd37c 100644 --- a/README.md +++ b/README.md @@ -21,6 +21,7 @@ then open a pull request. :zap: * Don’t leave trailing whitespace. * Not even leading indentation on blank lines. + #### Prefer implicit getters on read-only properties and subscripts When possible, omit the `get` keyword on read-only computed properties and @@ -76,6 +77,7 @@ internal struct TheFez { _Rationale:_ It's rarely appropriate for top-level definitions to be specifically `internal`, and being explicit ensures that careful thought goes into that decision. Within a definition, reusing the same access control specifier is just duplicative, and the default is usually reasonable. + #### Only explicitly refer to `self` when required When accessing properties or methods on `self`, leave the reference to `self` implicit by default: