-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Action to modify services #10010
Action to modify services #10010
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
CLAs look good, thanks! |
UI.user_error!("The key: '#{key}' is not supported in `enable_services' - following keys are available: [#{allowed_keys.join(',')}]") unless allowed_keys.include? key.to_sym | ||
end | ||
end), | ||
FastlaneCore::ConfigItem.new(key: :team_id, |
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.
We'd also need the team name here, like see produce/options.rb
👍
end | ||
|
||
def self.author | ||
"Bhimsen Padalkar" |
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.
Please use your Twitter or GitHub username 👍
def self.example_code | ||
[ | ||
'modify_services( | ||
username: "bhimsenp@thoughtworks.com", |
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.
Do you work at thoughtworks? Do you want to use a dummy email address instead?
This is great, that's a missing piece to use this produce feature from within fastlane 👍 I added a few comments before it's ready to be merged |
Incorporated the feedback |
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.
Nice, this is great. One quick question before we merge, have you tried running this action locally? Does it work as expected?
Yes it does.. I checked for few services for "on", "off" and other values (like "complete" for data_protection) |
Awesome, thanks for your contribution 👍 |
Hey @bhimsenpadalkar 👋 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 👍 |
It doesn't require immediate release. It can go in next release. |
Already live 🚀 Thanks again |
Congratulations! 🎉 This was released as part of fastlane 2.53.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
This is against the issue raised here
Description
This will add the action to modify the services.