-
Notifications
You must be signed in to change notification settings - Fork 36
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
Automate build and deployment #137
Comments
Same for iOS: opendocument-app/OpenDocument.ios#38 |
I usually create an unpublished GitHub release for each commit and attach the built AAR as a release asset. I'm sure there's a nicer way to do this. Would be possible to upload each build to GitHub repo's Maven, but each build would have to have it's version incremented, because I suspect that Maven wouldn't like duplicate version numbers.
Publish a new GitHub release and specify commit number in the general text field to choose which build to release, then CI would download APK from Maven and upload to Play Store and also attach to the published GitHub release. Unknowns for this approach:
|
Since this project is only an app, not a library, I don't think we need an AAR? Instead, why not store the APK as an artifact for each GitHub Action run? That way you won't have to increment the version number. I do that before a release manually anyway. Yes, we need to use Android App Bundles instead of APK. Not sure how that works from a CLI either (always build via Android Studio). Yes, we can public to Play Store on each GitHub release. I'm sure it's possible to publish to Play Store from GitHub Actions. I'll just need to issue the correct credentials for you. So without doing any research, I would imagine a workflow like this:
|
Alternatively, we could trigger the full process (build + deploy Android App Bundle + create GitHub release) on each push to master. That seems reasonable to me as I'm working on the development branch. It also avoids exceeding storage of GitHub Actions artifacts! |
Based on #136
The text was updated successfully, but these errors were encountered: