Doesn't work for guard-annotate anymore #160

Closed
jmuheim opened this Issue Jan 1, 2014 · 4 comments

Comments

Projects
None yet
2 participants

jmuheim commented Jan 1, 2014

I'd like to use guard-annotate to automatically run annotate when a file is changed. Sadly, I'm getting this error:

23:50:36 - INFO - Running annotate
/Users/josh/.rvm/gems/ruby-2.0.0-p353@transition/gems/annotate-2.6.0/bin/annotate:142:in `<top (required)>': invalid argument: --exclude tests,fixtures (OptionParser::InvalidArgument)
    from /Users/josh/.rvm/gems/ruby-2.0.0-p353@transition/bin/annotate:23:in `load'
    from /Users/josh/.rvm/gems/ruby-2.0.0-p353@transition/bin/annotate:23:in `<main>'
    from /Users/josh/.rvm/gems/ruby-2.0.0-p353@transition/bin/ruby_executable_hooks:15:in `eval'
    from /Users/josh/.rvm/gems/ruby-2.0.0-p353@transition/bin/ruby_executable_hooks:15:in `<main>'

I don't know where the problem lies (whether it's guard-annotate or annotate), and there's an issue opened for guard-annotate already here: cpjolicoeur/guard-annotate#12. Maybe you have an idea on how to fix it? The arguments seem valid, although annotate's complaining about them.

Thank you.

Owner

ctran commented Jan 1, 2014

Can you try this with 2.6.1 version (the latest version on rubygems.org)?

jmuheim commented Jan 1, 2014

This results in the same error:

00:46:42 - INFO - Running annotate
/Users/josh/.rvm/gems/ruby-2.0.0-p353@transition/gems/annotate-2.6.1/bin/annotate:143:in `<top (required)>': invalid argument: --exclude tests,fixtures (OptionParser::InvalidArgument)
    from /Users/josh/.rvm/gems/ruby-2.0.0-p353@transition/bin/annotate:23:in `load'
    from /Users/josh/.rvm/gems/ruby-2.0.0-p353@transition/bin/annotate:23:in `<main>'
    from /Users/josh/.rvm/gems/ruby-2.0.0-p353@transition/bin/ruby_executable_hooks:15:in `eval'
    from /Users/josh/.rvm/gems/ruby-2.0.0-p353@transition/bin/ruby_executable_hooks:15:in `<main>'
Owner

ctran commented Jan 2, 2014

Looks like this has been broken for a while. I'll check in a fix soon

ctran was assigned Jan 2, 2014

@ctran ctran pushed a commit that referenced this issue Jan 2, 2014

Cuong Tran Handle the "--exclude" option correctly, #152 and #160 6e80ed2

jmuheim commented Jan 2, 2014

Thanks a lot! 👍

@ctran ctran pushed a commit that referenced this issue Feb 26, 2014

Cuong Tran Merge branch 'release/2.6.2'
* release/2.6.2:
  v2.6.2
  Add known changes for 2.6.2.
  Retain existing position unless :force is passed
  Temporarily ignore integration tests
  Revert "Replacing old block with new one."
  Change email address
  Add .travis.yml
  switch deprecated "mock" for "double" in annotate specs
  use general file resolution for test files as well
  Replacing old block with new one.
  Fixed gemspec files attribute
  migrate.rake should live in lib/tasks
  Always load models, since they may not be autoloaded by Rails
  Handle the "--exclude" option correctly, #152 and #160
  FIX #157
  Pump version to 2.6.1 which fixes #22
ed327d5

ctran closed this Feb 26, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment