Rails 7.2 #786
Merged
Rails 7.2 #786
+1,298
−958
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
📦 esbuild Bundle Analysis for sakazukiThis analysis was generated by esbuild-bundle-analyzer. 🤖 |
yonta
force-pushed
the
feature/rails7.2/base
branch
4 times, most recently
from
3c25fb7 to
7080a83
Compare
|
mainにrebaseした |
fixture_path -> fixture_paths
警告内容は以下のようなもの。 DEPRECATION WARNING: Defining enums with keyword arguments is deprecated and will be removed in Rails 8.0. Positional arguments should be used instead
- rails app:updateコマンドを実行した - Rails設定のdiffを確認し、自分たちで設定した項目以外はデフォルトに追従した - rails db:migrateを実行した
- rails app:update を実行 - Rails設定のdiffを確認し、自分たちで設定した項目以外はデフォルトに追従した - rails generate コマンド実行後にRubocopフォーマットをかけるようにした
ファイルの変更があったときに検知して自動リロードする機能を提供していたgem。 Rails6時代はデフォルトで入っていた。 しかしRails7から、必須とは言えないためユーザが後から入れるものとしてデフォルトでなくなった。 SAKAZUKIでもERBファイルを編集したときには便利である。 しかしJS/TSなどの編集時には反応しないため、やはり手動リロードやbrowser syncが必要だった。 そのため、Railsデフォルトに従いSAKAZUKIからも削除することにする。
- bundle exec spring binstub --remove --all を実行 - Gemfileからspringを削除 - bundle install を実行
- rails app:updateを実行 - Rails設定のdiffに従い、絵文字設定のコメントアウト文を追加した
デフォルトですべてをオンにし、必要ないCopsをオフにするスタイルとした。
Rails 7.2系へアップデート
…ance Rubocop Performanceを追加
…config Rubocopの設定を一部変更
…-cookpad 新たなRubocopの設定にERBLintの設定を追従・修正した
Rubocop ERBを導入した
mainにマージされたサイズ機能で警告が起きていた。 内容は容量の数値がハードコード文字列になっているというもの。 これは言語差がないため、erblintのマジックコメントでdisableにした。
yonta
force-pushed
the
feature/rails7.2/base
branch
from
3fea695 to
78d3763
Compare
|
mainにrebase |
|
close #763 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
feature: rails7.2
完了したぞ！
やったこと