-
Notifications
You must be signed in to change notification settings - Fork 222
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
Native mobile app testing #227
Comments
Maybe worth creating new |
Why? I think one repo is much less cognitive load and seems reasonable to have it be able to test all 3 platforms using computer use. |
Ok One more question: do you think that is a good idea to spin up expo react native? we need env to run and test mobile Shortest tests |
Yeah not sure the approach but something like that for sure will be useful. |
Is someone/are people actively working on this? I'm interested in helping out. If not, I will start. |
@gladyshcodes has a draft PR #260. Probalby best to collaborate on that one, to reduce duplicate effort. |
Following up on our conversation with @rmarescu, we’ve decided to split #260 into several atomic PRs to reduce cognitive load, lower the risk of bugs, and ensure clearer, more focused code reviews. Thoughts: Ensuring SRP:
Scaling for mobile:
Testing:
Final:
Miscellaneous:
cc @slavingia |
I think we can discuss technical details next week, and scope for best approach. For now, we should pause development on this until we have a better foundation in place. |
@rmarescu, I remember you mentioned that AndroidInterface |
instead of another repo, wouldn't be better to have a seperated folder for mobile with its core functions just my draft https://github.com/anti-work/shortest/pull/230/files ? |
Yep! Just mapped out my thoughts on how pr should be split |
We need our own playground to test things. Regarding folder structure, we will most probably have a separate |
What
Why
The text was updated successfully, but these errors were encountered: