- Notifications
You must be signed in to change notification settings - Fork407
Fixes #1720 - move rule docs and update tests#1724
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
rjmholt commentedOct 4, 2021
Looks like platyPS invocations in the build script also need to be updated |
sdwheeler commentedOct 5, 2021
@rjmholt OK, I missed that. Will update. |
sdwheeler commentedOct 5, 2021
@rjmholt Fixed the build script. |
andyleejordan commentedOct 8, 2021
Hi,@rjmholt and@sdwheeler this PR caused a regression in the VS Code extension because the URL used to find the documentation was changed. Heads up to coordinate changes like this in the future. I'm pushing a new release today with the bug-fix. |
PR Summary
PR Checklist
Moved ./RulesDocumentation to ./docs/Rules
Moved ./docs/markdown to ./docs/Cmdlets
Update Pester test for documentation
PR has a meaningful title
Summarized changes
Change is not breaking
Make sure all
.cs,.ps1and.psm1files have the correct copyright headerMake sure you've added a new test if existing tests do not effectively test the code changed and/or updated documentation
This PR is ready to merge and is notWork in Progress.
WIP:to the beginning of the title and remove the prefix when the PR is ready.