-
Notifications
You must be signed in to change notification settings - Fork 74
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix the remaining issues with no spotify_shared.xcconfig #39
Conversation
8W9aG
commented
Jan 22, 2016
- Add some more options to spotify_os.xcconfig
- Make the target inherit these settings
- Fix some more test warnings
- Add a way to allow all certificates through code rather than macros
* builds. | ||
* @warning This will trigger an assert if all certificates are allowed on release builds. | ||
*/ | ||
@property (nonatomic, assign, readwrite, getter = isAllCertificatesAllowed) BOOL allCertificatesAllowed; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
areAllCertificatesAllowed
. But in this case, the getter is probably not important to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Will make the change
I re-triggered the failed build. Looked like the simulator flaking out. Fine now ✅ However @8W9aG seems like you’ll need to add some more unit tests 😄 Did notice that Coveralls isn’t able to find the source files from our repo. Not sure why though. See https://coveralls.io/builds/4818416/source?filename=SPTDataLoader%2FSPTDataLoaderService.m. Do you have any ideas why @jgavris? Seems like I’m not an admin at Coveralls so can’t look at the settings 😿 |
* Unsure why these didn't show up before
* Remove SPTDATALOADER_ALLOW_ALL_CERTS
bbe1ef2
to
1b68342
Compare
* The project.xcconfig will get overwritten by our build system when building for the Spotify iOS app * Meanwhile, on the open source build we will continue using the very aggressive spotify_os.xcconfig
👍 |
Fix the remaining issues with no spotify_shared.xcconfig
Revert "Merge pull request spotify#38 from peck/fix-for-disabled-headermaps"