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

Fix a typo in operator passed to Hash#key? #390

Merged
merged 1 commit into from
May 14, 2017
Merged

Fix a typo in operator passed to Hash#key? #390

merged 1 commit into from
May 14, 2017

Conversation

michaelbaudino
Copy link
Contributor

@michaelbaudino michaelbaudino commented May 12, 2017

I think this is just a typo introduced in 6102649, but it's causing the following error when trying to start any native_osx sync from master:

/Users/mike/code/docker-sync/lib/docker-sync/sync_strategy/native_osx.rb:55:in `key?': wrong number of arguments (0 for 1) (ArgumentError)
	from /Users/mike/code/docker-sync/lib/docker-sync/sync_strategy/native_osx.rb:55:in `start_container'
	from /Users/mike/code/docker-sync/lib/docker-sync/sync_process.rb:107:in `start_container'
	from /Users/mike/code/docker-sync/lib/docker-sync/sync_manager.rb:92:in `block in start_container'
	from /Users/mike/code/docker-sync/lib/docker-sync/sync_manager.rb:91:in `each'
	from /Users/mike/code/docker-sync/lib/docker-sync/sync_manager.rb:91:in `start_container'
	from /Users/mike/code/docker-sync/tasks/sync/sync.thor:163:in `daemonize'
	from /Users/mike/code/docker-sync/tasks/sync/sync.thor:47:in `start'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.4/lib/thor/command.rb:27:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.4/lib/thor/invocation.rb:126:in `invoke_command'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.4/lib/thor.rb:369:in `dispatch'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.4/lib/thor/base.rb:444:in `start'
	from bin/docker-sync:14:in `<main>'

@michaelbaudino michaelbaudino self-assigned this May 12, 2017
@EugenMayer
Copy link
Owner

arg, how did i miss this + how did i fail to test this ;/

@EugenMayer EugenMayer merged commit 0271633 into master May 14, 2017
@EugenMayer EugenMayer deleted the fix-typo branch May 14, 2017 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants