-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Update driver.go #454
Update driver.go #454
Conversation
Pull Request Test Coverage Report for Build 897
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR!
source/driver.go
Outdated
@@ -87,7 +87,7 @@ func Open(url string) (Driver, error) { | |||
d, ok := drivers[u.Scheme] | |||
driversMu.RUnlock() | |||
if !ok { | |||
return nil, fmt.Errorf("source driver: unknown driver %v (forgotten import?)", u.Scheme) | |||
return nil, fmt.Errorf("source driver: unknown driver '%v' (forgotten import?)", u.Scheme) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might as well make it '%s'
while you're at it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update!
I've stumbled upon the
source driver: unknown driver file (forgotten import?)
error which has been reported as an issue multiple times.This is a tiny proposal that might make the error message more readable:
instead of
source driver: unknown driver file (forgotten import?)
, the message can besource driver: unknown driver 'file' (forgotten import?)
to indicate that 'file' is the type of the driver