-
Notifications
You must be signed in to change notification settings - Fork 12
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
better help rendering #8
Comments
Closed
@wsmelton took care of a lot of this last night 🎉 ! Do you have a suggestion for https://docs.dbatools.io/#Export-DbaScript example 7? |
> is the key. See my previous PR on that huge multiline splat on
logshipping function
Il gio 4 ott 2018, 10:43 Chrissy LeMaire <notifications@github.com> ha
scritto:
… @wsmelton <https://github.com/wsmelton> took care of a lot of this last
night 🎉 ! Do you have a suggestion for
https://docs.dbatools.io/#Export-DbaScript example 7?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#8 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAHdB9os64YcrMp3EXJd1F10ityORucmks5uhcpMgaJpZM4Wy3bc>
.
|
okay, gonna fix it directly. |
9 tasks
We waiting to close this until we are finiished adding PS C:> or can we now? |
no, we can now, the general idea on how to write CBH and how the rendering goes has been "formalized" |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Managed to do this
but the problem/constraint is that multiline statements in the "commandline" of the help are not supported by CBH. So either we go for this representation and go correct the CBH on the few commands that have multiline examples, or we keep it the way they are right now...
e.g. of "mangled multiline"
The text was updated successfully, but these errors were encountered: