Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Delete Gemfile.lock from git and ignore it. #18

Merged
merged 1 commit into from May 30, 2012

Conversation

Projects
None yet
2 participants
Contributor

caius commented May 30, 2012

It shouldn't be committed for gem repos. Your gemspec should contain all the version requirements for gems ideally, so what's pinned in the Gemfile.lock is pretty irrelevant. (As I understand it.)

Yehuda wrote more on the subject, complete with a nice TL;DR to start the post off: http://yehudakatz.com/2010/12/16/clarifying-the-roles-of-the-gemspec-and-gemfile/

Delete Gemfile.lock from git and ignore it.
It shouldn't be committed for gem repos. Your gemspec should contain all the version requirements for gems ideally, so what's pinned in the Gemfile.lock is pretty irrelevant. (As I understand it.)

igrigorik added a commit that referenced this pull request May 30, 2012

Merge pull request #18 from EmberAds/remove-gemfile-lock
Delete Gemfile.lock from git and ignore it.

@igrigorik igrigorik merged commit 6623fa7 into igrigorik:master May 30, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment