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

Rails 7.1.4にあげた #787

Open
wants to merge 9 commits into
base: feature/rails7.2/base
Choose a base branch
from

Conversation

yonta
Copy link
Collaborator

@yonta yonta commented Nov 26, 2024

7.2に上げる前に、まずはパッチバージョンを上げる!

やったこと

  • bundlerアップデート
  • Railsのパッチバージョンを上げて7.1.4にした
    • rails app:updateに従い7.1.4の設定にアップデート
    • Rails 7.1のデフォルト設定で動作確認
    • Railsのデフォルト動作をv7.1にした
  • RSpec実行時の警告に従い、deprecatedな名前を変更
  • Rails 7.1系でデフォルト入らないgemを削除
    • listen gemを削除
    • spring gemを削除

メモ

  • なるべくRails 7.1にのみアップデート
    • 他gemのアップデートは最小限にした
  • 使うgemはRails 7.1系にはもともと上がっていたが、設定がされていなかった
    • デフォルト動作が7.0のままだった
    • 今回のアップデートで7.1系の動作を検証して、デフォルトも7.1に上げた
  • 設定ファイルのコメントのアップデートにもなるべく追従して、今後の差分を小さくするようにした。
  • listen
    • 開発中にファイル変更を監視してよしなにするgem
    • 開発が複雑化するためデフォルトオフ
  • spring
    • Railsの起動を高速化するgem
    • 現代PCだと効果が薄いためデフォルトオフ

格闘ログ

Issue #763

- rails app:updateコマンドを実行した
  - Rails設定のdiffを確認し、自分たちで設定した項目以外はデフォルトに追従した
- rails db:migrateを実行した
ファイルの変更があったときに検知して自動リロードする機能を提供していたgem。
Rails6時代はデフォルトで入っていた。
しかしRails7から、必須とは言えないためユーザが後から入れるものとしてデフォルトでなくなった。

SAKAZUKIでもERBファイルを編集したときには便利である。
しかしJS/TSなどの編集時には反応しないため、やはり手動リロードやbrowser syncが必要だった。

そのため、Railsデフォルトに従いSAKAZUKIからも削除することにする。
- bundle exec spring binstub --remove --all を実行
- Gemfileからspringを削除
- bundle install を実行
@yonta yonta changed the base branch from main to feature/rails7.2/base November 26, 2024 04:54
Copy link

📦 esbuild Bundle Analysis for sakazuki

This analysis was generated by esbuild-bundle-analyzer. 🤖
This PR introduced no changes to the esbuild bundle! 🙌

@yonta yonta changed the title まずRails7.1にあげた Rails 7.1にあげた Nov 26, 2024
@yonta yonta changed the title Rails 7.1にあげた Rails 7.1.4にあげた Nov 26, 2024
@yonta yonta self-assigned this Nov 26, 2024
@yonta yonta marked this pull request as ready for review November 26, 2024 05:13
@github-actions github-actions bot requested a review from momocus November 26, 2024 05:13
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.

1 participant