-
Notifications
You must be signed in to change notification settings - Fork 213
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
chore: added aab support and improved script #428
Conversation
780a9bf
to
65d6efa
Compare
aae87bb
to
ce82fac
Compare
scripts/prep-key.sh
Outdated
@@ -4,7 +4,7 @@ set -e | |||
export DEPLOY_BRANCH=${DEPLOY_BRANCH:-development} | |||
export PUBLISH_BRANCH=${PUBLISH_BRANCH:-master} | |||
|
|||
if [ "$TRAVIS_PULL_REQUEST" != "false" -o "$TRAVIS_REPO_SLUG" != "fossasia/badge-magic-android" ] || ! [ "$TRAVIS_BRANCH" == "$DEPLOY_BRANCH" -o "$TRAVIS_BRANCH" == "$PUBLISH_BRANCH" ]; then | |||
if [ "$TRAVIS_REPO_SLUG" != "fossasia/badge-magic-android" ] || ! [ "$TRAVIS_BRANCH" == "$DEPLOY_BRANCH" -o "$TRAVIS_BRANCH" == "$PUBLISH_BRANCH" ]; then |
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.
Revert so there is a common way of handling things, may remove the condition from .travis.yml
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.
sure, it will be consistent
scripts/upload-apk.sh
Outdated
|
||
#removing unused apps | ||
for file in app*; do | ||
if [[ ${file} =~ "unsigned" || ${file} =~ "unaligned" ]]; then | ||
rm ${file} | ||
elif [[ ${file} =~ ".aab" ]]; then |
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.
Master and dev aab should be separate
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.
Areeb, aab is a publishing format, hence it is of no use to the normal users. Why do you want to keep in the apk branch?
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.
that is the reason I just used it to push the app to playstore and keep the release and debug apk's in the apk branch.
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.
It is of use to me. Because I want to debug what was sent to play store if something goes wrong.
Normal users use fdroid or play store to download the APK. I and fellow developers and maintainers use the APK branch
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.
nvm, ill do it right away
scripts/upload-apk.sh
Outdated
fastlane supply --apk app.aab --track alpha --json_key fastlane.json --package_name $PACKAGE_NAME |
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.
master aab
scripts/upload-apk.sh
Outdated
@@ -46,6 +45,7 @@ git checkout --orphan temporary | |||
|
|||
# Add generated APK | |||
git add --all . | |||
git rm --cached app.aab |
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.
redundant
70a0c3c
to
f0e4b2a
Compare
f0e4b2a
to
747ac19
Compare
Fixes #427
Changes: