[DO NOT MERGE] CIチェック用ドラフト #774
Draft
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/spike
branch
from
b099eea to
6a65afa
Compare
yonta
force-pushed
the
feature/rails7.2/spike
branch
from
6a65afa to
8f36d07
Compare
fixture_path -> fixture_paths
- 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 を実行
警告内容は以下のようなもの。 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 generate コマンド実行後にRubocopフォーマットをかけるようにした
- rails app:updateを実行 - Rails設定のdiffに従い、絵文字設定のコメントアウト文を追加した
Rails 7.2系へアップデート
デフォルトですべてをオンにし、必要ないCopsをオフにするスタイルとした。
新規酒のsaveが失敗したときにもdatetimeの更新が呼ばれていた。 after_actionをやめて、酒のsaveが成功したときのみdatetimeを更新するようにした
createと同様にupdateのafter_actionをやめた。 このとき、酒の情報が更新されたかでflash_after_updateとupdate_datetimeがコンフリクトする。 これを正しい順番にし、コメントを追記しておいた。
デフォルトのEnforcedStyle: require_parenthesesなら必要ない。 EnforcedStyle: omit_parenthesesを設定してカッコ無しスタイルのときに必要な設定。
when/with/withoutのみで書く。
renderなどもカッコつきで呼び出すことを強制する。
yonta
force-pushed
the
feature/rails7.2/spike
branch
from
8f36d07 to
d014017
Compare
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.
No description provided.