-
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fastlane require format non plugins with dash #7998
Fastlane require format non plugins with dash #7998
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.
馃憤
This is awesome! Thanks @joshdholtz !! 馃殌 |
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 馃憤
|
||
def format_gem_require_name(gem_name) | ||
# from "fastlane-plugin-xcversion" to "fastlane/plugin/xcversion" | ||
gem_name = gem_name.tr("-", "/") if gem_name.start_with? "fastlane-plugin-" |
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.
Would replace "fastlane/plugin/" to "fastlane-plugin-" be more portable? E.g. if the plugin name has a dash in it?
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.
Ahhh, good call good call - that is probably a lot safer. @ohwutup, i think you mentioned you did work on the initial plugin stuff. Is this a better way of doing it? ^
Fixes #7502
fastlane_require "rest-client"
was formattingrest-client
asrest/client
was causing a 馃挜The comment above that logic was saying that replacing of
-
with/
was specifically forfastlane-plugin-
so now the logic inside offastlane_require
only replaces-
with/
if it starts withfastlane-plugin-
馃殌