-
Notifications
You must be signed in to change notification settings - Fork 15
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
I broke everything :P #22
Comments
Luckily the well implemented Unit- & UI-Tests advised you that there was something broken |
@PDDStudio what's a unit test? (kidding) |
@TheAndroidMaster I don't know lol.. You're the developer, not me /s |
@PDDStudio if you want to add some (please add some) go ahead! I probably won't get around to it (I don't know how to do it) for a while (I don't feel like learning how) because I have to study for school (because I want to watch Netflix). |
@TheAndroidMaster well, it depends on how the app is actually splitted in terms of modularity. If it's not ready-for-testing this might be a bunch of refactoring work lol Maybe adding some UI-Tests might be easier though |
When I was re-organizing a couple parts of the project, I took a few shortcuts that should probably be fixed before the next release. Mostly just issues with stuff being copied and pasted without paying attention. There are a couple random UI things (like menu buttons that don't open menus) that might need some work, but this should be a mostly easy fix. It just takes a while to find everything.
The text was updated successfully, but these errors were encountered: