rails commit log流し読み(2018/07/05)

2018/07/05分のコミットです。

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

---

A Class is a Module so we remove one conditional

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

LazyLoadHooks#execute_hookメソッドでbaseがClassかModuleかチェックするのに、Class、Moduleそれぞれに対してis_a?チェックしていたのを、Moduleに対してだけチェックするよう修正しています。

-            if base.is_a?(Class) || base.is_a?(Module)
+            if base.is_a?(Module)

ClassもModuleの為。

---

Remove vestigial require on ActiveStorage GCSService

activestorage/lib/active_storage/service/gcs_service.rbの修正です。

不要なactive_storage/filenameのrequireを削除しています。

---

Merge pull request #33220 from anniecodes/notes-command

railtiesの修正です。

rake taskとして実装されていたrails notesをrails command(普通のRubyのクラス)として実装しています。

これにより、annotation対象のタグを環境変数ではなくコマンドの引数として指定出来るようになっています。

$ ./bin/rails notes --help
Usage:
  bin/rails notes [options]

Options:
  -a, [--annotations=one two three]  # Filter by specific annotations, e.g. Foobar TODO
                                     # Default: ["OPTIMIZE", "FIXME", "TODO"]

---

Make NotesCommand tests more performant by getting rid of unecessary map

railties/test/commands/notes_test.rbの修正です。

notes コマンドのテストで不要なダミーデータを作成しないよう修正しています。