Skip to content
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

Beta release workflow #113

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Beta release workflow #113

wants to merge 4 commits into from

Conversation

fdocr
Copy link
Contributor

@fdocr fdocr commented Nov 16, 2020

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

Adds CD to our pipeline for a more streamlined release process

Related Tickets & Documents

#81

[optional] What gif best describes this PR or how it makes you feel?

go on - popcorn

@pr-triage pr-triage bot added the PR: draft label Nov 16, 2020
@CLAassistant
Copy link

CLAassistant commented Nov 16, 2020

CLA assistant check
All committers have signed the CLA.

@fdocr fdocr changed the title First test version Beta release workflow Nov 16, 2020
@fdocr fdocr marked this pull request as ready for review November 16, 2020 15:39
echo "This will fail because we don't have the secrets configured yet"
echo "$ANDROID_KEYSTORE_FILE" > keystore.jks.b64
base64 -d -i keystore.jks.b64 > app/keystore.jks
echo "storeFile=keystore.jks" >> keystore.properties
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The workflow from which this was based on (here) creates the keystore.properties manually, but we can just base64 encode it whole instead if that feels cleaner. In that case we can replace KEYSTORE_KEY_ALIAS, KEYSTORE_KEY_PASSWORD and KEYSTORE_STORE_PASSWORD for a single secret. I don't feel strongly about either approach

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should keep it separate. it will be easier to update

Copy link
Contributor

@maestromac maestromac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are blocked by Google's key limit :\

@fdocr
Copy link
Contributor Author

fdocr commented Nov 18, 2020

@maestromac yeah, until we can find a way to work around this I'm making this a Draft 👍

@fdocr fdocr marked this pull request as draft November 18, 2020 19:36
Base automatically changed from master to main January 18, 2021 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants