-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
docs: Document how to test on release builds #17952
Comments
picking up this one. |
Okay currently working on documenting how to test the app in release builds. I will include keystore setup, environment variable configuration, and troubleshooting tips. Let me know if there are any specific points you'd like me to cover! |
yeah , okay go for it , no problem |
Thanks @tarun1sisodia |
It may be that the best way for everyone to have the ability to test release builds is to include a "fallback-release-keystore.jks" file in perhaps the The creation of a keystore and configuration of a environment variables is the part of the process that is really annoying, in my opinion. Doing that would remove the need for 90% of the documentation stuff I wrote to Brayan |
Sure @mikehardy ,I'll be adding an fallback-release-keystore.jks file as a fallback when building a release variant. |
@mikehardy, added the suggested changes in this pr : #17961 |
The task is to document in the Wiki how to test the app in
release
builds. Try running the tests yourself first, then document it. If you develop any kind of script or add any tips, it would be tremendous.Mike's description in Discord should help:
That's a little annoying to set up but it's simply the price of entry for doing app signing
Anki-Android/.github/workflows/tests_emulator.yml
Lines 23 to 29 in 3b74b3d
What I do is I have a keystore setup, and I have a little script right next to it that exports all those vars
I should say it's not "easy", because there is some setup, but once you've got the keystore done and a little script that sets the env vars to match the keystore, then it's easy to iterate / do it whenever
That is is testable is kind of amazing, I don't even want to go into the details other than to say that it is kind of a crazy hack, and voczi deserves a lot of the credits, respect for that.
And that it is testable is the only way I was going to turn it on, fearing we'd have problems like what you're looking into in the future - there are always problems. But we can test them and fix them at least. Which unlocks like a 30MB app savings, so...on balance worth it
The text was updated successfully, but these errors were encountered: