なるようになるブログ

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

rails commit log流し読み(2018/11/11)

2018/11/11分のコミットです。

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


Prefer assert_not instead of refute [ci skip]

guides/bug_report_templates/generic_gem.rbguides/bug_report_templates/generic_master.rbの修正です。

bug report templatesでrefuteを使用していたのを、assert_notを使用するよう修正しています。contributing guideでassert_notを使用する事を推奨している為。

が、これは間違いで、bug report templatesではMinitest::Testを使用しているのですが、minitestではassert_notを使用出来ません(xx_notメソッドはRailsのテストクラスが独自に追加しているメソッドの為)。

これだとbug report templates実行するとエラーになってしまう為、後ほどrevertされています。


Revert "Merge pull request #34421 from albertoalmagro/use-assert-not-instead-of-refute"

という訳で直前のコミットをrevertしています。