-
Notifications
You must be signed in to change notification settings - Fork 90
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
Improve NoDownload Pretty instance on missing/unknown "mtarget" #960
Conversation
The guard was only used for the "Perhaps you meant"-part, so include the other part regardless of what `mtarget` is.
Thanks @runeksvendsen! Could you paste some example output? |
Yeah, example output would be nice, for both cases |
@hasufell @ffaf1 the new example output is the following.
|
Excellent. I would add quotes here too
(i.e. |
I found out how to trigger the other message:
Done: 6fec9d4 Output:
|
The guard was only used for the "Perhaps you meant"-part, so include the other part regardless of what
mtarget
is.Also: quote the version in the NotInstalled Pretty instance, so it matches that of the NoDownload Pretty instance.
Fixes #957