Граф коммитов

56 Коммитов

Автор SHA1 Сообщение Дата
Christopher Denter 8d7e6daafe Merge master -> no-force-unwrapping 2014-12-01 21:58:22 +01:00
Josh Abernathy 8e94807ae3 Merge pull request #21 from dennda/no-implicitly-unwrapped-optionals
Propose never using implicitly unwrapped optionals.
2014-12-01 13:18:51 -05:00
Christopher Denter 189a0d86ae Soften wording regarding implicitly unwrapped optionals (s/dont/avoid/) 2014-11-28 15:49:20 +01:00
Christopher Denter ee49df9581 Soften wording regarding force-unwrapping (s/dont/avoid/) 2014-11-28 15:43:19 +01:00
Christopher Denter fdc70f70e4 Propose not using force-unwrapping of optionals. 2014-11-25 20:52:10 +01:00
Christopher Denter bbb6e900d0 Propose never using implicitly unwrapped optionals. 2014-11-25 20:48:52 +01:00
Rob Rix 0f9a440bf0 Merge pull request #19 from dennda/master
Propose by-default usage of `let` over `var`.
2014-11-25 11:35:24 -05:00
Christopher Denter 62608d01e4 In -by-default section, remove reference to shadowing. 2014-11-25 13:14:17 +01:00
Christopher Denter 37abe2b73d Add a missing linebreak. (As per robrix) 2014-11-25 13:12:32 +01:00
Christopher Denter d7c75e0554 Propose by-default usage of `let` over `var`.
The intent and meaning of both keywords is clear, but *let-by-default*
results in safer and clearer code.
2014-11-24 12:19:56 +01:00
Justin Spahr-Summers d2ce27081a Merge pull request #18 from github/pr/17
Link to the Chinese version
2014-11-13 08:46:04 -08:00
joshaber 7d3321dcf8 Bullet. 2014-11-13 10:21:42 -05:00
joshaber 401fccb8cf Merge branch 'master' into pr/17
Conflicts:
	README.md
2014-11-13 10:21:10 -05:00
Justin Spahr-Summers d781b3f179 Merge pull request #16 from github/spaces-after-operator-definitions
Whitespace after operator names in def’ns.
2014-11-13 01:13:09 -08:00
Rob Rix 70b6b66d25 Editing 2014-11-13 01:08:00 -05:00
joshaber 9fbd83b7bb Separator. 2014-11-12 21:06:54 -05:00
joshaber bcedf19203 Move it to the bottom. 2014-11-12 21:05:31 -05:00
Artwalk b9453198b7 for pull request 2014-11-13 08:24:26 +08:00
Artwalk 0ef962ffc0 add a link to Chinese Version 2014-11-13 08:04:45 +08:00
Rob Rix d739150f52 Propose whitespace after operator names in def’ns. 2014-11-12 08:34:30 -05:00
Artwalk e97e3d22c8 translated to Chinese 2014-11-11 21:00:07 +08:00
Josh Abernathy e5539471bb Merge pull request #12 from github/omit-type-parameters
Propose omitting type parameters when possible.
2014-10-06 14:05:35 -04:00
Rob Rix b0f391f5a7 Propose omitting type parameters when possible. 2014-10-05 10:37:11 -04:00
Justin Spahr-Summers ed32b74d66 Merge pull request #10 from github/jspahrsummers-patch-2
Make as much of a class final as possible
2014-08-29 16:25:26 -07:00
Justin Spahr-Summers 8ad316b26e Rewrite as “make classes final by default,” expand it 2014-08-27 15:11:37 -07:00
Justin Spahr-Summers fe8c73ea47 Merge branch 'master' into jspahrsummers-patch-2
Conflicts:
	README.md
2014-08-27 15:10:42 -07:00
Rob Rix bcb758676d Merge pull request #8 from github/jspahrsummers-patch-1
Prefer structs over classes
2014-08-27 17:15:19 -04:00
Justin Spahr-Summers 28e54826b0 Replace “interface reuse” with “polymorphism” 2014-08-27 13:21:46 -07:00
Justin Spahr-Summers cf22f6b186 Make as much of a class final as possible 2014-08-26 13:02:14 -07:00
Justin Spahr-Summers faae2d091c Fill in “structs over classes” section 2014-08-26 12:59:23 -07:00
Justin Spahr-Summers fc6b1eb5ed Merge branch 'master' into jspahrsummers-patch-1 2014-08-26 12:46:01 -07:00
Justin Spahr-Summers 4568978d0a Merge pull request #9 from github/coolon
The colon belongs to the identifier
2014-08-26 11:33:59 -07:00
joshaber 7bd7da77d7 Rationale 2014-08-25 20:53:36 -04:00
joshaber 815bcc6d7c The One True Way 2014-08-25 20:38:19 -04:00
Justin Spahr-Summers e0571a5d01 Merge pull request #7 from github/no-trailing-whitespace
Don’t leave trailing whitespace section
2014-08-25 15:56:04 -07:00
Justin Spahr-Summers 94737f7711 Prefer structs over classes 2014-08-25 09:18:07 -07:00
Rob Rix e3028d2ee4 Space out, man. 2014-08-25 08:21:15 -04:00
Rob Rix 3599cabadd Update README.md 2014-08-25 08:20:44 -04:00
Rob Rix 1ec3bd4b3d Don’t leave trailing whitespace section
Brings in the no trailing whitespace section from https://github.com/github/objective-c-style-guide/pull/58.
2014-08-25 08:20:24 -04:00
Josh Abernathy 5fb7ea983c Merge pull request #6 from github/robrix-patch-1
Whitespace section from the ObjC style guide
2014-08-22 10:14:29 -04:00
Rob Rix 4721c6f259 Whitespace section from the ObjC style guide
Left out the trailing whitespace from https://github.com/github/objective-c-style-guide/pull/58 until we decide on that.
2014-08-22 09:59:03 -04:00
Justin Spahr-Summers e9b86c858e Merge pull request #5 from github/describe-purpose
Describe the purpose of and contribution steps for the style guide
2014-08-01 15:07:55 -07:00
Justin Spahr-Summers 0c7f387099 Add missing comma 2014-08-01 13:26:06 -07:00
Justin Spahr-Summers 3e10bf43b7 Add CONTRIBUTING 2014-08-01 13:25:11 -07:00
Justin Spahr-Summers 270673b580 What even is the point of a style guide? 2014-08-01 13:19:01 -07:00
Justin Spahr-Summers cdae9ae43a Merge pull request #4 from github/jspahrsummers-patch-1
Only explicitly refer to `self` when required
2014-08-01 13:07:53 -07:00
Justin Spahr-Summers 15613d4350 Update README.md 2014-08-01 12:25:26 -07:00
Justin Spahr-Summers 45c67efaa2 Only explicitly refer to `self` when required
(Also fixed spaces vs. tabs in another sample.)
2014-07-29 21:21:05 -07:00
Justin Spahr-Summers a5d2a34865 Merge pull request #3 from github/jspahrsummers-patch-1
When to include access control specifiers
2014-07-29 21:07:17 -07:00
Justin Spahr-Summers 728ce9e9bc When to include access control specifiers 2014-07-22 21:59:03 -07:00