rails commit log流し読み(2015/10/24) - なるようになるブログ

2015/10/24分のコミットです。

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

rails guideのThe Asset Pipelineの修正です。

In Productionの項、:cache、:concatオプションを等幅フォントで表示するよう"`"で囲むよう修正しています。

Move default uuid generation to active_record

migration生成時のprimary keyのtypeを任意の型に変更する設定(Set active_record config for always creating uuids in generators)を、config経由ではなくgenerators経由で行うよう修正しています。

# before
`Rails.application.config.active_record.primary_key = :uuid`

# after
config.generators do |g|
  g.orm :active_record, primary_key_type: :uuid
end

application/frameworkに関するオプションとgeneratorsに関するオプションを混ぜるべきではなく、generatorsに関するオプションはconfig.generatorsから指定出来るようにするのが正しいため、という理由のためのようです。

Use thor class_option to make the primary_key_type option work

Active Recordの修正です。

primary_key_typeをgeneratorsのオプションと指定出来るようにする為に、primary_key_typeをThorのclass_optionメソッドで指定するよう修正しています。

Stop aligning the arguments

activerecord/lib/rails/generators/active_record/model/model_generator.rbの修正です。

class_optionメソッドを呼び出す際に、オプションの指定位置を揃えるためにあった不要なスペースを削除、及びHashを1.9記法で書くよう修正しています。

Remove skip on tests that have been fixed.

actionpack/test/controller/parameters/mutators_test.rbの修正です。

Hash#select!メソッドの挙動がCRubyとJRubyで異なっている箇所がある影響で、JRubyでのみ動作しないテストがあり、元々はJRubyの場合のみskipするようになっていたのですが、JRuby側でCRubyと同じ挙動をするよう対応(Fixes #3137. Hash#select! inconsistency with MRI when yielding only o… · jruby/jruby@303a128)されたので、JRubyでも該当箇所のテストを行うようskipを削除しています。

Merge pull request #22040 from arunagw/remove-jruby-travis-opts

.travis.ymlの修正です。

travisの設定からJRubyのオプションを削除しています。現在はJRubyのテストを実行していないため、オプションを指定する必要は無い為削除したとの事です。