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
[regression][pilot] Fix upload using api_key_path + apple_id
CLI options
#18860
[regression][pilot] Fix upload using api_key_path + apple_id
CLI options
#18860
Conversation
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.
Hear me out here... maybe the should_login
argument on start()
should only apply to Apple ID login 🤔
The reason that exists is because of avoiding 2FA stuff that was super annoying on CI and whatnot. So maybe we revert this change in BuildManager
and instead...
- Login in
start
if there is a token - If no token and
should_login
, then we attempt to login with Apple ID
I think that is essentially what this is doing but we should maybe move that to the start
/ login
methods 🤷♂️
Yes, this is it. this should be the case! You are 💯 right 💪
Okay, I think, i require your help in this PR...don't want to break our lovely pilot 😇 Do you want to take over, or shall I try first? 🙇 |
@crazymanish Ha! You can start if you'd like and then I can add a commit if I see anything that needs to change 🤷♂️ Otherwise I don't mind taking over from here! Your call 😊 |
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.
One small thought otherwise this looks good!
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.
I really like this solution! 😊 Thanks for reworking this ❤️
Hey @crazymanish 👋 Thank you for your contribution to fastlane and congrats on getting this pull request merged 🎉 Please let us know if this change requires an immediate release by adding a comment 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.
Congratulations! 🎉 This was released as part of fastlane 2.186.0 🚀
Checklist
bundle exec rspec
from the root directory to see all new and existing tests passbundle exec rubocop -a
to ensure the code style is validMotivation and Context
api_key_path + apple_id
CLI optionsapi_key_path + apple_id
input options #18771 (comment)Description
spaceship token
is nil and input options haveapi_key_path
Testing Steps
Added the unit-tests for this new functionality 😇
Update Gemfile and run
bundle install
withusing CLI
with api_key_path and apple_id