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

add watch os as target to decompose #156

Merged
merged 2 commits into from
May 28, 2021
Merged

add watch os as target to decompose #156

merged 2 commits into from
May 28, 2021

Conversation

plusmobileapps
Copy link
Contributor

@plusmobileapps plusmobileapps commented May 27, 2021

Overview

try to setup watchOS support by following similar pattern to tvOS

Questions

I would like to add a sample for watchOS before merging, but would we need to also update MVI Kotlin to support watchOS? Or just good enough to merge as is and create another issue for making a sample?

Issue

Issue #155

Copy link
Contributor

@arkivanov arkivanov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also add the new target to the build and publish workflows.

@arkivanov
Copy link
Contributor

And also please add the badge to the README :-)

@arkivanov
Copy link
Contributor

Regarding watchos sample, I don't think it is really necessary, but we can discuss it separately. We also don't have samples for macos or tvos. There are tests, they should verify if everything is alright.

…e readme and documentation for supported targets
@arkivanov arkivanov merged commit b8313ac into badoo:master May 28, 2021
@arkivanov arkivanov mentioned this pull request May 28, 2021
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.

2 participants