Skip to content
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

Lazy load xcode-select and xcodebuild #27

Merged
merged 4 commits into from
Mar 31, 2018
Merged

Lazy load xcode-select and xcodebuild #27

merged 4 commits into from
Mar 31, 2018

Conversation

joshdholtz
Copy link
Contributor

Motivation

We recently add simctl to fastlane (thank you for this awesome gem 馃槉) in this PR - fastlane/fastlane#12039. However, we had to do some weird things to make our rubocop setup happy when including this gem on different operating systems. The initial execution of finding xcode-select and xcodebuild on gem/file load causes weird things when running on a non-macOS.

My proposed change lazy loads the finding of xcode-select and xcodebuild to prevent issues of when this gem is included in things that aren't always running on macOS. 馃挭

Let me know if there is anything else you would like added to this PR or if you have any questions regarding my thoughts/issues 鉂わ笍

@coveralls
Copy link

coveralls commented Mar 29, 2018

Coverage Status

Coverage decreased (-0.01%) to 95.117% when pulling f64e1a9 on joshdholtz:master into 5545516 on plu:master.

Copy link
Owner

@plu plu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks!

@plu plu merged commit bdb6732 into plu:master Mar 31, 2018
@joshdholtz
Copy link
Contributor Author

@plu Thank you! Can you push a new release to RubyGems at some point when you are some free time? 馃檭

@plu
Copy link
Owner

plu commented Apr 2, 2018

@joshdholtz On it...

@joshdholtz
Copy link
Contributor Author

@plu Thank you! Bumping the the fastlane dependency now to 1.6.3. Thanks again for this great gem! 鉂わ笍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants