Skip to content
This repository has been archived by the owner on Jun 17, 2020. It is now read-only.

.rbenv-version, .ruby-version, .rvmrc #111

Merged
merged 1 commit into from Nov 8, 2013
Merged

Conversation

teamon
Copy link
Contributor

@teamon teamon commented Oct 24, 2013

Should we add them to the .gitignore, or keep one in repo? To be honest, I'm confused what's the best way, and which path we want to follow.

@teamon @sheerun @Ostrzy @szajbus @chytreg

@szajbus
Copy link
Contributor

szajbus commented Mar 19, 2013

Let's discuss.

Arguments for keeping it in the repo:

  • there's no default way to freeze ruby version for a project (like it is with gems)
  • if default ruby version changes, this would force devs to upgrade
  • allows to work on upgrade to newer ruby version in separate branch and have main branch working on older version of ruby

Arguments against:

  • too many redundant files (.rbenv-version, .ruby-version, .rvmrc), need to keep them in sync!
  • need to notify other devs about ruby version upgrade

@jandudulski
Copy link
Contributor Author

Alternative is to add ruby version in Gemfile and leave to dev preferences which ruby version manager to use.

@chytreg
Copy link
Member

chytreg commented Mar 19, 2013

There is another problem rbenv via rvmrc. Do we want to force of using one of ruby manager? I'm to lazy when use rbenv to keep update the rvmrc file which I'm not using.

Gemfile alternative is ok, but this is not the case. We should have the information e.g. in README file wich ruby we use for the project and that's all.

@sheerun
Copy link
Contributor

sheerun commented Mar 19, 2013

Don't include any of these files. Add ruby version to Gemfile.

@jandudulski
Copy link
Contributor Author

That's Adam way - do not argue. Just command.

❤️ ❤️ ❤️

@szajbus
Copy link
Contributor

szajbus commented Mar 19, 2013

Damn. I didn't know that you can specify this dependency in Gemfile!

Then do it, and .gitignore these files.

Michał Szajbe

On Tuesday, March 19, 2013 at 3:08 PM, Dariusz Gertych wrote:

There is another problem rbenv via rvmrc. Do we want to force of using one of ruby manager? I'm to lazy when use rbenv to keep update the rvmrc file which I'm not using.
Gemfile alternative is ok, but this is not the case. We should have the information e.g. in README file wich ruby we use for the project and that's all.


Reply to this email directly or view it on GitHub (#111 (comment)).

@sheerun
Copy link
Contributor

sheerun commented Mar 19, 2013

If you want reason, .ruby-version and others force specific patch version.

@sheerun
Copy link
Contributor

sheerun commented Mar 19, 2013

rubygems/bundler#2228

@teamon
Copy link
Contributor

teamon commented Mar 19, 2013

Do NOT specify ruby version in gemfile (I want to run 1.9 apps locally with 2.0 since it's much faster)
Do NOT add .rvmrc, .ruby-version, .rbenv-version to git, put them in gitignore
Specify ruby version in README

@bartoszrega
Copy link
Member

@teamon 👏

@szajbus
Copy link
Contributor

szajbus commented Mar 20, 2013

@teamon kinda makes sense. But why not move the app to 2.0 in prod as well? ;]

@teamon
Copy link
Contributor

teamon commented Mar 20, 2013

Soon, but not yet.

Tymon Tobolski

On Wednesday, March 20, 2013 at 11:06 AM, Michał Szajbe wrote:

@teamon (https://github.com/teamon) kinda makes sense. But why not move the app to 2.0 in prod as well? ;]


Reply to this email directly or view it on GitHub (#111 (comment)).

@sheerun
Copy link
Contributor

sheerun commented Mar 20, 2013

Bundler 2.0 is going to allow for ruby '>= 1.9.3' in Gemfile.

@teamon
Copy link
Contributor

teamon commented Mar 20, 2013

Is going == is isn't now. And it would require gem bundler update everywhere since it's a breaking change...

Tymon Tobolski

On Wednesday, March 20, 2013 at 11:16 AM, Adam Stankiewicz wrote:

Bundler 2.0 is going to allow for ruby '>= 1.9.3' in Gemfile.


Reply to this email directly or view it on GitHub (#111 (comment)).

@Ostrzy
Copy link
Contributor

Ostrzy commented Mar 29, 2013

For now ruby version should be specified in README.
If bundler 2.0 come out and it supports feature referenced by @sheerun, than we can just switch to that.

@sheerun sheerun closed this in 9fe662c Mar 30, 2013
@teamon
Copy link
Contributor

teamon commented Oct 2, 2013

For new projects that runs on 2.0 put 2.0 in Gemfile. It will help for example circleci discovering required ruby version

@teamon teamon reopened this Oct 2, 2013
@jandudulski
Copy link
Contributor Author

👍

teamon added a commit that referenced this pull request Nov 8, 2013
.rbenv-version, .ruby-version, .rvmrc
@teamon teamon merged commit 88abf4e into master Nov 8, 2013
@teamon teamon deleted the feature/ruby-2.0-gemfile branch November 8, 2013 00:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants