なるようになるブログ

読書感想文かrailsについてかrubyについてか

rails commit log流し読み(2019/10/07)

2019/10/07分のコミットです。

CHANGELOGにのったコミットは以下の通りです。

actionpack/CHANGELOG.md

activemodel/CHANGELOG.md


Fixed pluralization typo

rails guideのAction View Form Helpersの修正です。

Dealing with Basic Formsの項のグラマーの修正を行っています。


Updated ActionDispatch::Request.remote_ip=

actionpack/lib/action_dispatch/http/request.rbの修正です。

ActionDispatch::Request#remote_ip=で値を更新後、ActionDispatch::Request#remote_ipで値を参照した際に更新前の値が返ってくるバグがあったのを修正しています。


fix(fork-tracker): Only patch fork if it's supported

activesupport/lib/active_support/fork_tracker.rbactivesupport/test/fork_tracker_test.rbの修正です。

forkメソッドが使えない環境で、fork向けの拡張を適用しないよう修正しています。


Document ActionView::UrlHelper in overview guide

rails guideのAction View Overviewの修正です。

ActionView::UrlHelper moduleで提供しているメソッドについての説明を追加しています。


Merge pull request #37325 from jaredbeck/patch-6

rails guideのActive Job Basicsの修正です。

Exceptionsの項にexceptionをraiseしてrescueされない場合そのjobは失敗と見なされる旨説明を追加、及び、Retrying or Discarding failed jobsの項に特に設定が無い場合失敗したjobはリトライされない旨を追加しています。


Less Pure Copping: cop-out tmp folders for generated Rails apps

.rubocop.ymlの修正です。

railtiesのテストで生成されるdummy appsをRubocopの対象から除外するようにしています。


[ci skip] switch eg. to proper e.g.

docの修正です。

eg.e.g.にまとめて修正しています。


Merge pull request #37215 from utilum/avoid_test_flunking_on_warning

railtiesのテストの修正です。

assert_equalでstdoutの出力結果をチェックしていた箇所を、assert_matchを使用するよう修正しています。

Ruby 2.7でwarningが出てしまっていて、warningが含まれていてもassertionが通るようにする為。


Merge pull request #37268 from jonathanhefner/dummy-app-dynamic-rails-version

railties/lib/rails/generators/rails/app/templates/config/application.rb.ttの修正です。

pluginのテスト用のdummy appsの場合、config.load_defaultsに固定の文字列ではなくRails::VERSION::STRING.to_fで取得した値を設定するよ修正ています。

pluginのRailsのバージョンを上げた際に、自動で最新のconfigを設定出来るようにする為。


Merge pull request #37217 from joshmn/frozen_attributes

activemodel/lib/active_model/attributes.rbの修正です。

インスタンス自体をfreezeしている場合に、ActiveModel::Attributesで指定したattributeが変更出来るようになっていたのを、freezeされている場合は変更出来ない(FrozenErrorをraiseする)よう修正しています。


Merge pull request #37100 from jonathanhefner/fix-irb-tab-completion-warning

activesupport/lib/active_support/deprecation/proxy_wrappers.rbの修正です。

hashinstance_methodsメソッドについてはdeprecateメッセージを表示せずにtargetdelegateするよう修正しています。

IRBで上記メソッドを使用しており、IRBでタブ補完をする際にdeprecateメッセージが表示されないようにする為。


Remove unused requires

actionview/lib/action_view/lookup_context.rbactiverecord/lib/active_record/associations.rbの修正です。

使用していないrequireを削除しています。


Add SECURITY.md file, linking to security policy

security policyについて記載した.github/security.mdを追加しています。

詳細な説明はファイル内では行わず、security policyへのリンクとなっています。


Merge pull request #37381 from tekin/document-caveats-in-autosave

activerecord/lib/active_record/autosave_association.rbのdocの修正です。

ActiveRecord::AutosaveAssociationのdocに、既に保存済みのassociationの場合の挙動についての注記を追加しています。


Update comment in controller generators

railties/lib/rails/generators/rails/scaffold_controller/templates/api_controller.rb.ttrailties/lib/rails/generators/rails/scaffold_controller/templates/controller.rb.ttの修正です。

controller generatorsのコメントに、[ci skip] change all instances of blacklist and whitelist to denylist…で修正した"white list"という言い回しが残っていたのを削除しています。


Merge pull request #37094 from p8/fix-autosave-comment

activerecord/lib/active_record/autosave_association.rbのdocの修正です。

nested_records_changed_for_autosave?メソッドのdocにメソッドが既に呼ばれている場合falseを返す旨説明を追加しています。