Skip to content

Use another style guide? #3

Description

@modocache

How about we just use https://github.com/github/swift-style-guide or https://github.com/raywenderlich/swift-style-guide? That way someone else can do all the work of maintaining it, and we just have to follow it. 😉

I prefer the GitHub one.

Activity

  1. sharplet commented on May 4, 2015

    @sharplet

    Just had a read, and it looks pretty good. I especially like the bit about 2-space indent, but that's probably impractical for this org. 😉

    Speaking of which, we should probably add something about indentation settings being set explicitly in the Xcode project, so that contributors don't accidentally commit code with their potentially different personal indentation settings.

    One other thing — I think the one about protocol conformance in separate extensions may be fine for application targets, but I think for framework authors it's problematic due to not being able to export public extensions in a module.

  2. paulyoung commented on May 4, 2015

    @paulyoung

    I basically follow the GitHub one anyway 👍

  3. modocache commented on May 4, 2015

    @modocache
    MemberAuthor

    I especially like the bit about 2-space indent, but that's probably impractical for this org.

    I wouldn't be opposed to doing two giant pull requests to get Quick and Nimble in conformance with whatever we decide on.

    Speaking of which, we should probably add something about indentation settings being set explicitly in the Xcode project, so that contributors don't accidentally commit code with their potentially different personal indentation settings.

    Definitely!

    One other thing...

    Yeah, I think the best approach here is to put a link to an existing style guide, plus a few bullet points for exceptions. I'm pretty opposed to maintaining our own style guide (why reinvent the wheel?). Thoughts, @jeffh? Heck, we could even use the Pivotal Labs style guide, provided it's public. 😉

  4. jeffh commented on May 5, 2015

    @jeffh
    Member

    Sure, the github style guide works for me. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions