-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat(actions) Add additional encrypt/decrypt actions. Allow session create to use different users. #23
Conversation
This PR is tackling two very important issues. 1) A complex config for the consuming app's Fastlane setup 2) A way to authorize a session and then later login with different accounts.
Can we please update the readme to include which |
lib/fastlane/plugin/react_native_release/actions/add_app_var.rb
Outdated
Show resolved
Hide resolved
lib/fastlane/plugin/react_native_release/actions/decrypt_app_vars.rb
Outdated
Show resolved
Hide resolved
tag_prefix = 'releases' | ||
base_branch = params[:beta_branch] | ||
target_branch = params[:production_branch] | ||
# TODO: ohter branches |
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.
ohter
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.
will address in #24
sh "git show-ref #{target_branch}" do |status| | ||
sh "git branch -D #{target_branch}" if status.success? | ||
end | ||
# if !is_hotfix |
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.
TODO: hotfix
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.
lol didnt mean to do markdown here
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.
added as #24
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.
looks good to me. Maybe some additional comments in the
TODO: flows
section to help jog anyone's memory when they come back there
nice work!
|
||
def self.authors | ||
# So no one will ever forget your contribution to fastlane :) You are awesome btw! | ||
["cball", "isaiahgrey93"] |
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.
👍
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.
look forward to increasing this array size 😄
is_beta = target.include?('beta') | ||
is_hotfix = params[:hotfix] === true | ||
release_type = params[:release_type] || UI.select("Select a release type:", VALID_RELEASE_TYPES) | ||
is_beta = release_type.include?('beta') |
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.
Not important at all, but these three lines (and the following conditional) could be done metaprogram-y to set the state for more release types in the future.
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.
totally agree. I want to rework this whole action in a followup PR to properly support master-based and gitflow-based workflows (and hotfixes). I'll try and clean up there as well.
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.
My Fastlane skill is wayyyy out of date, but FWIW looks amazing. Two small unimportant comments.
This PR is tackling two very important issues.
Changes
create_fastlane_session
. It now re-uses some ENV vars, and prompts for the user to release as.add_app_var
action to add a single env vardecrypt_android_keystore
andgenerate_android_keystore
to make working with keystores easierencrypt_app_vars
anddecrypt_app_vars
to handle writing / restoring ENV.decrypt_app_vars
takes an optional namespace which allows us to override foralpha
,beta
, orrelease
encrypt_google_play_credentials
anddecrypt_google_play_credentials
to handle writing/restoring the Google Play JSON