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.

Sign up for GitHub

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

Jump to bottom

railsを7.0.8から7.2にupdateする #94

Merged
merged 5 commits into from
Sep 11, 2024
Merged

railsを7.0.8から7.2にupdateする #94

merged 5 commits into from
Sep 11, 2024

Conversation

kakeru-one
Copy link
Contributor

@kakeru-one kakeru-one commented Sep 10, 2024 •
edited
Loading

対応内容

  • railsを7.2にupdateさせました。
    • それに伴って依存しているgemもupdateさせた。
  • Rackをupdateしたので、それに伴ってテストコードを修正した。
    • Rack::Utils::SYMBOL_TO_STATUS_CODEの422のsymbolが変わったので対応させる必要があったため
  • Deprecate ActiveSupport::ProxyObjectに対応するためにjbuilderをupdateする

別issueで対応すること

@kakeru-one kakeru-one self-assigned this Sep 10, 2024
@kakeru-one kakeru-one changed the title Update rails to 7 2 railsを7.0.8から7.2系にupdateする Sep 10, 2024
@kakeru-one kakeru-one changed the title railsを7.0.8から7.2系にupdateする railsを7.0.8から7.2にupdateする Sep 10, 2024
@kakeru-one kakeru-one added the enhancement New feature or request label Sep 10, 2024
@kakeru-one kakeru-one marked this pull request as ready for review September 10, 2024 09:03
@kakeru-one kakeru-one requested a review from kuri0616 September 10, 2024 09:03
@kuri0616
Copy link
Collaborator

こちらもUPDATEありがとうございます！
ステータスコードのシンボルが変わるとは。なかなか、影響の大きな変更があったんですね💦

ProxyObject
があまりよく理解できないのですが(メソッドの呼び出しを委譲する？？)
これをJ builderが使用していたが、非推奨になった。
ProxyObjectを使わない仕様に変更したものに知るためUpdateしたって感じですかね？

@kakeru-one
Copy link
Contributor Author

これをJ builderが使用していたが、非推奨になった。
ProxyObjectを使わない仕様に変更したものに知るためUpdateしたって感じですかね？

合っています！！

kuri0616
kuri0616 reviewed Sep 11, 2024
View reviewed changes
Copy link
Collaborator

@kuri0616 kuri0616 left a comment

Choose a reason for hiding this comment

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

LGTM！(僕が理解しただけ😂)

@kuri0616 kuri0616 merged commit 11eefdd into main Sep 11, 2024
1 check passed
kakeru-one added a commit to sns-archive/sns-archive-viewer that referenced this pull request Sep 22, 2024
@kakeru-one
railsを7.0.8から7.2にupdateする
bfcab0e 
Ref: Progaku-copy/progaku-archive#94
@kakeru-one kakeru-one mentioned this pull request Sep 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kuri0616 kuri0616 kuri0616 left review comments

Assignees

@kakeru-one kakeru-one

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kakeru-one @kuri0616