Skip to content
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

Bump command ref to 5.4.0 #247

Merged
merged 3 commits into from
Jan 11, 2023
Merged

Bump command ref to 5.4.0 #247

merged 3 commits into from
Jan 11, 2023

Conversation

fflaten
Copy link
Collaborator

@fflaten fflaten commented Jan 10, 2023

Fresh command reference docs based on the new Pester 5.4.0 release.
This includes a example changes and adds missing parameter-help, especially Should.mdx.

Fix pester/Pester#2128
Related pester/Pester#2170

docs/commands/Should.mdx Outdated Show resolved Hide resolved
@fflaten
Copy link
Collaborator Author

fflaten commented Jan 10, 2023

Needs a review, especially Should.mdx now that it's rendered and easier to have an opinion on the parameter help. Should we change the placeholder text (besides the \< \> glitch atm)?

Depends on operator being used. See Get-ShouldOperator -Name \<Operator\> for help.

@bravo-kernel
Copy link
Collaborator

Is <_<MethodName in https://github.com/pester/docs/blob/711d87076d113ca5742047d36bed52b671438201/docs/commands/New-MockObject.mdx correct?

Other than that, and the two parser bugs, this looks good to merge to me.

@bravo-kernel bravo-kernel merged commit db7b5bb into pester:master Jan 11, 2023
@bravo-kernel
Copy link
Collaborator

Thanks @fflaten and happy to see that there were only a few bumps with the parser 🎉

@fflaten fflaten deleted the pester54 branch January 11, 2023 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing Parameter Docs in V5
2 participants