-
Notifications
You must be signed in to change notification settings - Fork 511
Fix markdown syntax #1382
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 markdown syntax #1382
Conversation
this makes markdown linter ignore duplicate headers in the same document
* have all indentation to look alike * have code of snippets to use `\t`
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.
LGTM! Thanks for cleaning this up!
docs/community_snippets.md
Outdated
], | ||
"description": "Pester Test for Parameter" | ||
} | ||
"Pester for Mandatory Pester": { |
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.
Should this be Pester for Mandatory Parameter
?
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.
I guess so, based on the prefix and h3.
But I didn't change any content to the snips, as they are not mine.
(#1297)
Let me know if it is desired to fix that with this PR. I don't mind
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.
Yeah, that was just a typo from the original PR. Would you mind fixing it? Thanks.
@tylerl0706 : I didn't do the |
@lipkau if you can, that'd be great. Otherwise, it's ok |
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.
LGTM
Fixed the |
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.
LGTM 👍
PR Summary
\t
PR Checklist
Note: Tick the boxes below that apply to this pull request by putting an
x
between the square brackets.Please mark anything not applicable to this PR
NA
.WIP:
to the beginning of the title and remove the prefix when the PR is ready