なるようになるブログ

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

rails commit log流し読み(2018/10/22)

2018/10/22分のコミットです。

CHANGELOGへの追加はありませんでした。


Rename event variable

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

Subscriber#startメソッドでActiveSupport::Notifications::Event classのインスタンスを格納する変数名をeからeventに変更しています。


Correct some tests related to changes in #33079 (#34272)

railties/test/generators/app_generator_test.rbrailties/test/generators/shared_generator_tests.rbの修正です。

generatorのテストでJSに関するチェックが古いままになっていた(webpackerをデフォルトにした対応の修正漏れ)のを修正しています。


Remove :javascript from Rails::PluginBuilder::PASSTHROUGH_OPTIONS

railties/lib/rails/generators/rails/plugin/plugin_generator.rbの修正です。

Rails::PluginBuilder::PASSTHROUGH_OPTIONSからもう使用されていないjavascriptオプションを削除しています。


Remove javascripts and javascript_engine options for generators

railties/lib/rails/generators.rbの修正です。

Rails::Generators::DEFAULT_OPTIONSからもう使用されていないjavascriptsjavascript_engineオプションを削除しています。


Merge pull request #34274 from bogdanvlviv/remove-yarn-files-from-gitignore-template

railties/lib/rails/generators/rails/app/templates/gitignore.ttの修正です。

rails newで生成する.gitignoreからyarnに関するファイルを削除しています。同じ内容をwebpacker側で追加する為、とのこと。


Merge pull request #34279 from bogdanvlviv/remove-extra-remove_file-skip_action_cable

railties/lib/rails/generators/rails/app/app_generator.rbの修正です。

rails new--skip_action_cableオプションを指定した場合のapp/javascript/channels/consumer.jsファイルの削除処理を削除しています。その後でapp/javascript/channelsディレクトリ自体を削除している為。