X Tutup
The Wayback Machine - https://web.archive.org/web/20210122031927/https://github.com/radex/SwiftyTimer/issues/21
Skip to content
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

Convert separate test project into a XCTest-based test target #21

Open
radex opened this issue Feb 29, 2016 · 5 comments
Open

Convert separate test project into a XCTest-based test target #21

radex opened this issue Feb 29, 2016 · 5 comments
Labels

Comments

@radex
Copy link
Owner

@radex radex commented Feb 29, 2016

PR-s appreciated.

@radex radex added the help wanted label Feb 29, 2016
@Austinate
Copy link
Contributor

@Austinate Austinate commented Feb 29, 2016

During this change i'd think it would be good to make 1 target instead multiple we have now (what do you think?).
Also we should keep in mind that test target won't be available for watchOS (i think it's fine since same code can be tested with any other)

@radex
Copy link
Owner Author

@radex radex commented Feb 29, 2016

During this change i'd think it would be good to make 1 target instead multiple we have now (what do you think?).

My understanding is that you need separate targets to build frameworks for separate platforms. If you can get it to work with Carthage on a single target, that would be great, though.

@Austinate
Copy link
Contributor

@Austinate Austinate commented Feb 29, 2016

I've seen this approach in Mapper.
So i think we shouldn't have issues with that

@radex
Copy link
Owner Author

@radex radex commented Feb 29, 2016

Cool — you're more than welcome to give it a try ;)

@Austinate
Copy link
Contributor

@Austinate Austinate commented Feb 29, 2016

i see what you're saying, will do soon ;)

@radex radex mentioned this issue Feb 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
X Tutup