Skip to content

Add basic watchos support #441

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

Merged
merged 5 commits into from
Jun 10, 2019
Merged

Add basic watchos support #441

merged 5 commits into from
Jun 10, 2019

Conversation

lammertw
Copy link
Contributor

Depends on mac-cain13/R.swift.Library#28

Add a Watch app and extension using R.swift to the sample application.

@lammertw
Copy link
Contributor Author

I was wondering about the state of this PR. Is it being considered for inclusion. We're now using my fork in our projects and would love to go back to the non forked release.

@tomlokhorst
Copy link
Collaborator

Hi, thank you for your PR, and sorry for the (very) late reply.

We are interested in adding watchOS support, and merging this PR, but we've been very busy these last few months. When we've finally figured out a good fix for the issues with the New Build System we'll have some time to look add merging new features.

Again, thank you for this PR!

@teameh
Copy link

teameh commented May 27, 2019

Any updates on this? We've quite a big project stuck on the 5.0.0 alpha release branch from @lammertw because we need watchOS support. Anything I can do to help get this merged :) ?

@tomlokhorst
Copy link
Collaborator

No updates, other than that we did manage to figure out the build system issues and R.swift 5 has been released (5.0.3 now).

I do plan on looking at this PR (and others), but after WWDC, once we know if there's major changes in the next Xcode.

@tomlokhorst tomlokhorst merged commit 72a3f89 into mac-cain13:master Jun 10, 2019
@tomlokhorst
Copy link
Collaborator

Thanks again for this PR!
Sorry it took so long, we were busy with other stuff.

I've merged it and it will be part of the next R.swift release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants