search

tpope / vim-rails

rails.vim: Ruby on Rails power tools
http://www.vim.org/scripts/script.php?script_id=1567
4.1k stars 383 forks source link

Support rake notes task #567

Closed eloyesp closed 4 years ago

eloyesp commented 4 years ago

I've just found that when running :Rails notes the errorformat does not seems to match those notes. It does not seems to be a super important adittion, and I'm not sure how difficult could be doing that (I've been reading about the errorformat without much luck, I can try a pull-request myself if someone tell me that the format makes it possible).

The format of the message is:

app/models/post.rb:
  * [ 67] [TODO] suffers :reek:DuplicateMethodCall and :reek:FeatureEnvy

app/models/board.rb:
  * [ 10] [TODO] should do other stuff instead

app/models/todo.rb:
  * [ 16] [TODO] abstract this out
  * [ 54] [TODO] missing test for uniq call

Regards.

eloyesp commented 4 years ago

I've just checked and it seems to work fine when run like: :Rake notes

tpope commented 4 years ago

This should really be fixed to work for :Rails notes too.

tpope commented 4 years ago

P.S. Rails can go fuck itself for using such an asinine format.