-
Notifications
You must be signed in to change notification settings - Fork 487
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 .aab installation support #103
Add .aab installation support #103
Conversation
lib/util/bundletool.js
Outdated
module.exports = function(options) { | ||
return new Promise(function(resolve, reject) { | ||
var defaultBaseDir = ('/tmp/bundletool') | ||
var bundletoolUrl = 'https://api.github.com/repos/google/bundletool/releases/latest' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMO bundletool should be bundled with docker image.
STF may be deployed in various environments which may not have network access.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but your branch is outdated.
Could you merge/rebase on latest changes?
Of course, I'll update this branch on latest changes |
Signed-off-by: nghia.viminh <nghia.viminh@gameloft.com>
Signed-off-by: nghia.viminh <nghia.viminh@gameloft.com>
…dletool binary in storage-temp Signed-off-by: nghia.viminh <nghia.viminh@gameloft.com>
Signed-off-by: nghia.viminh <nghia.viminh@gameloft.com>
e57b982
to
ee3e00c
Compare
I've rebased the branch, you can merge the changes. Thank you |
Drag & drop installation and launching of .aab files. Set your own keystore options to sign APKs in storage-temp unit