Skip to content

New-MarkdownHelp - Fix #720#724

Closed
Jaykul wants to merge 1 commit intopester:masterfrom
Jaykul:PlatyPS
Closed

New-MarkdownHelp - Fix #720#724
Jaykul wants to merge 1 commit intopester:masterfrom
Jaykul:PlatyPS

Conversation

@Jaykul
Copy link
Copy Markdown
Contributor

@Jaykul Jaykul commented Feb 6, 2017

Create a branch for PlatyPS help to address #720

Initially, just import New-MarkdownHelp -Module Pester -OutputFolder Docs -WithModulePage

I recommend we not merge to DevelopmentV4, but create a branch. Before merging, we should get a Build to run PlatyPS on the source markdown to build the xml help, and then retrofit all the functions to reference the xml files instead of having comments...

@it-praktyk
Copy link
Copy Markdown
Contributor

Generally I agree with your idea to move help but

  1. Markdown based help is planned for the next release
  2. Pull requests what wait for merging can improve Pester 4.0.x release too - specially that include są new features

@Jaykul
Copy link
Copy Markdown
Contributor Author

Jaykul commented Feb 7, 2017

Yeah, like my fixes for gherkin in #715 ;-)

@it-praktyk
Copy link
Copy Markdown
Contributor

Good job! BTW, The PassThru parameter in the Gherkin function hasn't help yet.

BTW, you did partially similar job why I did #691. @nohwnd , could you merge waiting pull requests?

@dlwyatt dlwyatt changed the base branch from DevelopmentV4 to master March 30, 2017 14:52
@it-praktyk it-praktyk changed the title New-MarkdownHelp #720 New-MarkdownHelp - Fix #720 Aug 23, 2017
@it-praktyk it-praktyk added this to the v4.1 milestone Sep 2, 2017
@Jaykul Jaykul closed this Nov 3, 2017
@Jaykul Jaykul deleted the PlatyPS branch November 3, 2017 04:42
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.

2 participants