Skip to content
This repository has been archived by the owner on Feb 7, 2018. It is now read-only.

can't dup NilClass #98

Closed
tae8838 opened this issue Jun 27, 2017 · 0 comments
Closed

can't dup NilClass #98

tae8838 opened this issue Jun 27, 2017 · 0 comments

Comments

@tae8838
Copy link

tae8838 commented Jun 27, 2017

I get TypeError: can't dup NilClass when doing
Cocaine::CommandLine.new('some command', nil).run
Could we have an error message that's more descriptive like a Cocaine error that says "nil is not a valid argument"?

jyurek pushed a commit to thoughtbot/terrapin that referenced this issue Feb 2, 2018
`Terrapin::CommandLine.new` had a defualt 2nd and 3rd argument of "" and {},
respectively. But if you explicitly pass in `nil`, then you'll get an error
because `#dup` isn't on `nil`. It's not immediately appareny why you get this
error, so this change makes it so that we tell you.

Fixes thoughtbot/cocaine#98
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant