wbailey / command_line_reporter

A gem for making it easy to produce a report while a ruby script is executing
Apache License 2.0
432 stars 23 forks source link

Warnings when using command_line_reporter #25

Open eirvandelden opened 8 years ago

eirvandelden commented 8 years ago

Running ruby with warnings yields the following warnings:

/Users/me/Documents/tool/vendor/bundle/ruby/2.1.0/gems/command_line_reporter-3.3.6/lib/command_line_reporter/formatter/progress.rb:10: warning: `*' interpreted as argument prefix
/Users/me/Documents/tool/vendor/bundle/ruby/2.1.0/gems/command_line_reporter-3.3.6/lib/command_line_reporter/formatter/progress.rb:33: warning: method redefined; discarding old indicator
/Users/me/Documents/tool/vendor/bundle/ruby/2.1.0/gems/command_line_reporter-3.3.6/lib/command_line_reporter/formatter/nested.rb:36: warning: method redefined; discarding old message_string
/Users/me/Documents/tool/vendor/bundle/ruby/2.1.0/gems/command_line_reporter-3.3.6/lib/command_line_reporter/formatter/nested.rb:40: warning: method redefined; discarding old complete_string
/Users/me/Documents/tool/vendor/bundle/ruby/2.1.0/gems/command_line_reporter-3.3.6/lib/command_line_reporter/formatter/nested.rb:44: warning: method redefined; discarding old indent_size
/Users/me/Documents/tool/vendor/bundle/ruby/2.1.0/gems/command_line_reporter-3.3.6/lib/command_line_reporter/column.rb:27: warning: method redefined; discarding old size
wbailey commented 8 years ago

Thanks for submitting. Just confirming this was in ruby 2.1.0?

eirvandelden commented 8 years ago

Yes. 2.1.1 to be precise :)

Op 26 jan. 2016 om 20:17 heeft Wes Bailey notifications@github.com het volgende geschreven:

Thanks for submitting. Just confirming this was in ruby 2.1.0?

— Reply to this email directly or view it on GitHub.

tvanbolderen commented 7 years ago

This happens on Ruby 2.3.0 as well. I need to suppress the warnings by running Ruby with -W0

./vendor/bundle/ruby/2.3.0/gems/command_line_reporter-3.3.6/lib/command_line_reporter/formatter/progress.rb:10: warning: `*' interpreted as argument prefix
./vendor/bundle/ruby/2.3.0/gems/colored-1.2/lib/colored.rb:45: warning: shadowing outer local variable - value
./vendor/bundle/ruby/2.3.0/gems/command_line_reporter-3.3.6/lib/command_line_reporter/formatter/progress.rb:33: warning: method redefined; discarding old indicator
./vendor/bundle/ruby/2.3.0/gems/command_line_reporter-3.3.6/lib/command_line_reporter/formatter/nested.rb:36: warning: method redefined; discarding old message_string
./vendor/bundle/ruby/2.3.0/gems/command_line_reporter-3.3.6/lib/command_line_reporter/formatter/nested.rb:40: warning: method redefined; discarding old complete_string
./vendor/bundle/ruby/2.3.0/gems/command_line_reporter-3.3.6/lib/command_line_reporter/formatter/nested.rb:44: warning: method redefined; discarding old indent_size
./vendor/bundle/ruby/2.3.0/gems/command_line_reporter-3.3.6/lib/command_line_reporter/column.rb:27: warning: method redefined; discarding old size
wbailey commented 7 years ago

Thanks for pointing this out. I'll have a look