-
Notifications
You must be signed in to change notification settings - Fork 84
Fixes for Xcode 7 #156
Fixes for Xcode 7 #156
Conversation
briancroom
commented
Sep 4, 2015
- Switch to Cedar's Xcode7 branch for now
- Update the test bundles to use XCTest
- Use a helper WatchKit 1 app target to build the storyboard fixture for the WatchKit specs
35f16b3 to
219e5cc
Compare
- Some issues with nullability - Turn back deployment targets to silence some NSURLConncection deprecation warnings
I ended up needing to regenerate them using the XCTest bundle templates since the migrator didn’t work properly.
- Introduce a WatchKit app target to convince Xcode to compile the storyboards - Remove the “Rootless” storyboard and related spec because Xcode no longer compiles unreachable interface controller in storyboards
When using CedarApplicationDelegate, the specs are executed before XCTest execution begins, which makes Xcode treat the test run as a failure.
ea7427c to
37e9f60
Compare
|
This has now been rebased and tested with Xcode 7 final. I know a couple folks are waiting for this - any objections to merging? |
|
Please, let's merge On Tue, Sep 22, 2015 at 2:47 PM, Brian Croom notifications@github.com
|
|
When will the version of PivotalCoreKit be updated so that we can use the XCode7 fixes with cocoapods? Should we be pointing to master in the mean time? |
|
I looked into this briefly today. It looks like the project needs some updates for running one or more of the test suites from the command line before we can cut a new *green release. For the moment master is probably the right place to be. Please continue to send your pr's and/or issues. Thanks! On Thu, Oct 8, 2015 at 3:40 PM, David Rodriguez notifications@github.com
|