Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Running SSHKit tests with Rake 11.0 reveals many Ruby warnings #344

Open
mattbrictson opened this issue Apr 22, 2016 · 2 comments
Open

Running SSHKit tests with Rake 11.0 reveals many Ruby warnings #344

mattbrictson opened this issue Apr 22, 2016 · 2 comments
Labels

Comments

@mattbrictson
Copy link
Member

Rake 11.0 turns on Ruby warnings by default. With warnings enabled, running SSHKit's test suite results in a flood of messages like this:

lib/sshkit/runners/group.rb:15: warning: instance variable @group_size not initialized
lib/sshkit/host.rb:59: warning: instance variable @local not initialized
lib/sshkit/backends/abstract.rb:141: warning: instance variable @pwd not initialized
lib/sshkit/backends/abstract.rb:149: warning: instance variable @env not initialized
lib/sshkit/backends/abstract.rb:149: warning: instance variable @user not initialized
lib/sshkit/backends/abstract.rb:149: warning: instance variable @group not initialized

We could:

  • Treat these warnings as legitimate and try to fix them; or
  • Turn off Ruby warnings

I haven't had time to think this through yet, so for now I'm just opening it for discussion.

Your thoughts?

@leehambley
Copy link
Member

Warnings are legit, we should solve them, but if it's only a test problem, I wouldn't let it block you from 1.10 release?

@mattbrictson
Copy link
Member Author

Nope, this is definitely not a blocker. Just wanted to make note of it before I forgot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants