-
-
Notifications
You must be signed in to change notification settings - Fork 476
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 should help examples #1647
Fix should help examples #1647
Conversation
Very nice PR mr. @asears. I personally really appreciate these kind of changes for their educational value. I've added one remark relating to the multi line code examples, they should be fenced or they will not render properly on the pester.dev website. https://docusaurus-powershell.netlify.app/docs/faq/multi-line-examples |
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.
Few suggestions.
Co-authored-by: Jakub Jareš <[email protected]>
Co-authored-by: Jakub Jareš <[email protected]>
Co-authored-by: Jakub Jareš <[email protected]>
Co-authored-by: Jakub Jareš <[email protected]>
Fence code examples, split link examples
Add definition
Updates to test parameters documentation
Co-authored-by: Jakub Jareš <[email protected]>
clarify local runs of test.ps1
LGTM, @asears are you done with all your changes? |
Yes will open another PR if I have further improvements. |
Signed-off-by: asears [email protected]
1. General summary of the pull request
Improves #1634