redmine-trunk-changes 2019-08-04
rubocopの設定の修正やリファクタリングがありました。
rubocop周り雑に--safe-auto-correct
かけてしまってよいのでは、という気がしなくもない。
特定のCopだけかけるなら--only
で指定できるし。
- Auto-correct - RuboCop: The Ruby Linter that Serves and Protects
- RuboCopで導入される`--safe`オプションと`--safe-auto-correct`オプション - koicの日記
パッチなげてみようかな
リビジョン 18342 - rubocop: use Layout/TrailingWhitespace AllowInHeredoc true - Redmine
heredoc中の末尾の空白をありに設定変更しています。 これ1ファイルしかないのでそのままtodo.ymlの内容をrubocopに移動する、でもよさそうなような?(あんまりheredoc中で末尾に空白書きたくなることなさそうなので)
リビジョン 18343 - rubocop: fix Layout/BlockAlignment of test/unit/lib/redmine/scm/adapters/merc... - Redmine
ブロックを開始する行のインデントを直しています。
リビジョン 18344 - code clean up test/unit/lib/redmine/scm/adapters/mercurial_adapter_test.rb - Redmine
ブランチ名の変数がbraと省略されていたのでbranchにリネームしています。
リビジョン 18345 - rubocop: fix Layout/CommentIndentation of test/unit/repository_cvs_test.rb - Redmine
コメントのインデント位置を修正しています。
リビジョン 18346 - Previous and next month links in gantt (#31373). Patch by Mizuki ISHIKAWA. - Redmine
ガントチャートに前の月、次の月へのリンクを追加しています。 ApplicationHelperの方にリンクの生成メソッドが移動しています。 これってアプリケーション全体で使えるのかな? あとで使い方見てみよう
リビジョン 18347 - rubocop: remove app/helpers/calendars_helper.rb from Style/UnneededInterpolat... - Redmine
rubocopでExcludeしていたファイルがすでに直っていたのでtodoから消しています。
リビジョン 18348 - rubocop: fix Layout/DefEndAlignment of test/unit/repository_cvs_test.rb - Redmine
def endのendのインデントがずれてたのを直しています。 endの前のアサーションがそもそもインデントずれていますね
リビジョン 18349 - rubocop: fix Layout/IndentationConsistency of test/unit/repository_cvs_test.rb - Redmine
そのendの前のアサーションのインデントを直しています。
リビジョン 18350 - rubocop: fix Layout/DefEndAlignment of test/functional/documents_controller_t... - Redmine
endのインデントずれを直しています。
全ファイルでLayout/DefEndAlignment
が効くようになったみたいです👏
リビジョン 18351 - rubocop: fix Style/YodaCondition of lib/redmine/scm/adapters/abstract_adapter.rb - Redmine
yodaコンディション(定数的なものを左におく)を直しています
これself.
いらなさそうだな