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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TIMOB-23526] Locks down the version of cocoapod hyperloop uses. #35

Closed
wants to merge 1 commit into from
Closed

Conversation

Sophrinix
Copy link

@Sophrinix Sophrinix commented Jun 15, 2016

JIRA: https://jira.appcelerator.org/browse/TIMOB-23526

This locks down the version of cocoapods that we are using and makes the pod command run through bundler which is a good practice in the ruby world.

While this doesn't actually solve the problem of using a version of cocoapod newer than 0.39.0, it does however allow cocoapods to be used again for hyperloop projects.
Not having a Gemfile when you have a ruby dependency is akin to not having a package.json in the node world.

It's good practice in the ruby world to use bundler when calling ruby commands aka bundle exec pod... as opposed to pod ....

This buys us time in the short run and in the longer run makes the ruby dependencies more predictable.

…the pod command run through bundler which is a good pratice in the ruby world
@hansemannn
Copy link
Contributor

Hey @Sophrinix, I'm considering to use this solution for now, so people can at least use it out of the box. One question outstanding is, if this will change the version only for Hyperloop or globally. So in detail, if people want to use CocoaPods 1.x for other projects, would this PR prevent them? If not, we can go ahead. /cc @cheekiatng

@Sophrinix
Copy link
Author

This will only change the version of cocoapods for that mobile app. bundler is local to a given project.

@hansemannn
Copy link
Contributor

Love it. Can you update to the latest version before 1.x? I think it was 0.39.0. Will merge afterwards

@@ -0,0 +1,62 @@
GEM
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this lock-file need to be part of the commit?

@hansemannn
Copy link
Contributor

Closing in favor of #49 to be able to support both.

@hansemannn hansemannn closed this Aug 3, 2016
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

2 participants