I did all steps for this project (and even the dependency GCDKit, see here) to work as a tvOS framework and tried to apply the same structure as with the other targets. It's building fine for me so I think this can be merged after a short review. Shouldn't break anything.
I have experienced an issue with the tvOS test target though. I think it's because on tvOS the document paths are a little different. This should be solved but can also be done after merging this – I suggest merging this first and create an issue afterwards. Maybe this pull request form AERecord helps solving the issue, looks similar to me.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.
## 📋 Pull Request Information
**Original PR:** https://github.com/JohnEstropia/CoreStore/pull/38
**Author:** [@Jeehut](https://github.com/Jeehut)
**Created:** 1/17/2016
**Status:** ✅ Merged
**Merged:** 1/19/2016
**Merged by:** [@JohnEstropia](https://github.com/JohnEstropia)
**Base:** `master` ← **Head:** `feature/tvOS_support`
---
### 📝 Commits (2)
- [`9322371`](https://github.com/JohnEstropia/CoreStore/commit/9322371224a37514bfdb33df4054795443560901) Use tvOS_support feature branch of GCDKit (Carthage + git submodules)
- [`9081b36`](https://github.com/JohnEstropia/CoreStore/commit/9081b36cca31a9a164b307221f52e7ea9af643d2) Add tvOS target + Configure target + Add shared scheme for tvOS
### 📊 Changes
**6 files changed** (+447 additions, -5 deletions)
<details>
<summary>View changed files</summary>
📝 `.gitmodules` (+1 -1)
📝 `Cartfile` (+1 -1)
📝 `Cartfile.resolved` (+1 -1)
📝 `Carthage/Checkouts/GCDKit` (+1 -1)
📝 `CoreStore.xcodeproj/project.pbxproj` (+344 -1)
➕ `CoreStore.xcodeproj/xcshareddata/xcschemes/CoreStore tvOS.xcscheme` (+99 -0)
</details>
### 📄 Description
I did all steps for this project (and even the dependency GCDKit, see [here](https://github.com/JohnEstropia/GCDKit/pull/8)) to work as a tvOS framework and tried to apply the same structure as with the other targets. It's building fine for me so I think this can be merged after a short review. Shouldn't break anything.
I have experienced an issue with the tvOS test target though. I think it's because on tvOS the document paths are a little different. This should be solved but can also be done after merging this – I suggest merging this first and create an issue afterwards. Maybe [this](https://github.com/tadija/AERecord/pull/19) pull request form AERecord helps solving the issue, looks similar to me.
---
<sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
📋 Pull Request Information
Original PR: https://github.com/JohnEstropia/CoreStore/pull/38
Author: @Jeehut
Created: 1/17/2016
Status: ✅ Merged
Merged: 1/19/2016
Merged by: @JohnEstropia
Base:
master← Head:feature/tvOS_support📝 Commits (2)
9322371Use tvOS_support feature branch of GCDKit (Carthage + git submodules)9081b36Add tvOS target + Configure target + Add shared scheme for tvOS📊 Changes
6 files changed (+447 additions, -5 deletions)
View changed files
📝
.gitmodules(+1 -1)📝
Cartfile(+1 -1)📝
Cartfile.resolved(+1 -1)📝
Carthage/Checkouts/GCDKit(+1 -1)📝
CoreStore.xcodeproj/project.pbxproj(+344 -1)➕
CoreStore.xcodeproj/xcshareddata/xcschemes/CoreStore tvOS.xcscheme(+99 -0)📄 Description
I did all steps for this project (and even the dependency GCDKit, see here) to work as a tvOS framework and tried to apply the same structure as with the other targets. It's building fine for me so I think this can be merged after a short review. Shouldn't break anything.
I have experienced an issue with the tvOS test target though. I think it's because on tvOS the document paths are a little different. This should be solved but can also be done after merging this – I suggest merging this first and create an issue afterwards. Maybe this pull request form AERecord helps solving the issue, looks similar to me.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.