-
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
Rename shield_io_timeout to shield_service_timeout #11478
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.
|
I signed it! |
CLAs look good, thanks! |
The contents of the # Autogenerated by fastlane
#
# Ensure this file is checked in to source control!
source "https://rubygems.org"
gemspec path: File.expand_path("/Users/jenkins/fastlane")
plugins_path = File.join(File.dirname(__FILE__), 'fastlane', 'Pluginfile')
eval_gemfile(plugins_path) if File.exist?(plugins_path)
gem 'xcpretty' And the contents of # Autogenerated by fastlane
#
# Ensure this file is checked in to source control!
gem 'fastlane-plugin-badge', path: "/Users/jenkins/fastlane-plugin-badge" |
Thanks for the PR, really appreciated. This shouldn't have happened, and this PR will also break all fastlane setups using that parameter, I'll reach out to the package author 👍 |
@KrauseFx so should I go ahead and close this PR? |
Seems like it's resolved as soon as a new version by Daniel is pushed 👍 |
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 commit changed the name of this parameter: HazAT/badge@cdf190d
Our CI at work failed to run fastlane because the parameter name change caused an error.
After making the changes here and this associated change the build was able to get past the previous error. However now I'm running into a different error when trying to run
scan
orgym
:I'm not sure if this is because of how I'm loading the forked fastlane and plug-in repos, or if it's an actual issue.
Description
This updates the name from
shield_io_timeout
toshield_service_timeout
to match