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

Rubyのバージョンを 2.5.0 へ上げる #32

Merged
merged 2 commits into from
Mar 12, 2018

Conversation

pokotyamu
Copy link
Contributor

@pokotyamu pokotyamu commented Mar 6, 2018

Block

  • e-Navigator チームの合意が取れている
  • マージした後に告知いる?
    • 告知はしない。新しく始める人が 2.5.0 でやってもらう

やりたかったこと

  • Ruby のバージョンアップ

やったこと

  • .ruby-version で 2.5.0 を指定
  • README の更新

確認方法

  • 差分確認

@pokotyamu pokotyamu self-assigned this Mar 6, 2018
README.md Outdated
- Bundler
- PostgreSQL

Ruby のバージョンは、2.5.0(より最新)でなければならない理由はありませんが、なるべく最新を使うように心がけてください。
Copy link
Contributor Author

Choose a reason for hiding this comment

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

わりと、2.4.3 じゃないとダメなんですか?的な質問来てたので、 README に一文追加した。
必要なければ消しちゃう。

Copy link
Contributor

Choose a reason for hiding this comment

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

この1文はなくても良さそうに思います!
その質問は受け続けたいというか...調査的な意味を込めて。

Copy link
Contributor

Choose a reason for hiding this comment

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

個人的には書かなくてもいいような気もしているけどなんとも言えない… 🤔
質問してきた人たちは他の古いバージョンの Ruby を使ってて新たに入れるの面倒とかそういう人?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

質問してきた人たちは他の古いバージョンの Ruby を使ってて新たに入れるの面倒とかそういう人?

おそらく、それ系な人だと思う!(そもそもバージョン管理的に Ruby を入れていない可能性もあるけど...)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

消す方向で修正する 🙋‍♂️

@hanocha
Copy link
Contributor

hanocha commented Mar 6, 2018

現在進行中の人たちはそのまま進めてもらって、この後始める人達に関して 2.5.0 にするみたいな感じかな 🤔
マージ後の全体告知とかは特にいらないような気もしている

@sukechannnn
Copy link
Contributor

告知は僕も特に必要ないと思います🙏

@sukechannnn
Copy link
Contributor

あと、ついでにGemfileのBundlerのバージョンをアップデートしておいてもらえると助かります🙏

@pokotyamu
Copy link
Contributor Author

GemfileのBundlerのバージョンをアップデート

@kielze これってどーゆーこと?ごめん理解力がないだけかも 🙏

@sukechannnn
Copy link
Contributor

sukechannnn commented Mar 6, 2018

あ、Gemfile.lock をgit管理から消したから関係ないやつでした!
すみません🙇

@pokotyamu
Copy link
Contributor Author

最終的に ruby-version の更新と、最低限の README の更新だけになった 🏎

Copy link
Contributor

@sukechannnn sukechannnn left a comment

Choose a reason for hiding this comment

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

LGTMです!

@pokotyamu
Copy link
Contributor Author

マージしちゃう!

@pokotyamu pokotyamu merged commit dfa1414 into master Mar 12, 2018
@pokotyamu pokotyamu deleted the feature/bump-ruby-2.5.0 branch March 12, 2018 09:04
mikutas pushed a commit to mikutas/e-navigator that referenced this pull request Jun 15, 2020
Rubyのバージョンを 2.5.0 へ上げる
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants