Rolf Xcode 8 API review

that can be done in several ways (we won’t be able to cover all of them), including reviewing the API diff to stuff that looks weird

• Review tasks:

  • Review api definition & custom code for:

    • FIXMEs / TODOs
    • obsolete
    • stuff that looks weird
    • long objc-looking names
    • extraneous models
    • non-internal fields
    • async candidates
    • designated initializers
    • candidates for abstract types

  • Review api diff for:

    • stuff that looks weird
    • long objc-looking names
    • obsolete
    • extraneous models
    • non-internal fields
    • async candidates

https://gist.github.com/spouliot/ed4d466d733dfe1ee4b21b4317e370a1

might be worth looking into the API added in tvOS

• ExternalAccessory PR

• HomeKit PR

• MultipeerConnectivity PR

  • Review api definition & custom code for:

    • FIXMEs / TODOs
    • obsolete
    • stuff that looks weird
    • long objc-looking names
    • extraneous models
    • non-internal fields
    • async candidates
    • designated initializers
    • candidates for abstract types

  • Review api diff for:

    • stuff that looks weird
    • long objc-looking names
    • obsolete
    • extraneous models
    • non-internal fields
    • async candidates

• Photos

• PhotosUI PR

  • Review api definition & custom code for:
    • FIXMEs / TODOs
    • stuff that looks weird
    • long objc-looking names
    • obsolete
    • extraneous models
    • non-internal fields
    • async candidates
    • designated initializers
    • candidates for abstract types
  • Review api diff for:
    • stuff that looks weird
    • long objc-looking names
    • obsolete
    • extraneous models
    • non-internal fields
    • async candidates

• ReplayKit PR 1 PR 2

  • Review api definition & custom code for:

    • FIXMEs / TODOs
    • obsolete
    • stuff that looks weird
    • long objc-looking names
    • extraneous models
    • non-internal fields
    • async candidates
    • designated initializers
    • candidates for abstract types

  • Review api diff for:

    • stuff that looks weird
    • long objc-looking names
    • obsolete
    • extraneous models
    • non-internal fields
    • async candidates

• UserNotification PR 1 PR 2

  • Review api definition & custom code for:

    • FIXMEs / TODOs
    • obsolete
    • stuff that looks weird
    • long objc-looking names
    • extraneous models
    • non-internal fields
    • async candidates
    • designated initializers
    • candidates for abstract types

  • Review api diff for:

    • stuff that looks weird
    • long objc-looking names
    • obsolete
    • extraneous models
    • non-internal fields
    • async candidates

• VideoSubscriberAccount PR 1 PR 2

  • Review api definition & custom code for:

    • FIXMEs / TODOs
    • stuff that looks weird
    • long objc-looking names
    • obsolete
    • extraneous models
    • non-internal fields
    • async candidates
    • designated initializers
    • candidates for abstract types

  • Review api diff for:

    • stuff that looks weird
    • long objc-looking names
    • obsolete
    • extraneous models
    • non-internal fields
    • async candidates