Skip to content

task UpdateCBH: Bug fixed in updating Comment Based Help in functions to point to external help - #10

Merged
zloeber merged 4 commits into
ModuleBuild:masterfrom
schittli:master
Mar 31, 2018
Merged

task UpdateCBH: Bug fixed in updating Comment Based Help in functions to point to external help#10
zloeber merged 4 commits into
ModuleBuild:masterfrom
schittli:master

Conversation

@schittli

Copy link
Copy Markdown

The Issue: If we have multiple <# … #> Comments like the following situation, then everything between the first <# and the last #> was replaced by the single one external help link:

Function Set-FileContent-Encoded() {
	<#
	.SYNOPSIS
		…
	#>
	[CmdletBinding()]
	Param (
		…
	)

	Function ConvertHashtableTo-Object {
		<#
		.SYNOPSIS
			…
		#>
		…
	}
}

The Fix:

  • only affects this file
  • recognizes multiple Comment Based Help texts <# … #> correctly and replaces only the first found Comment Based Help texts with the link to the external help.

SmartGit@jig.ch added 4 commits March 29, 2018 14:14
Bug fixed in updating Comment Based Help in functions to point to external help
The Issue: If we have multiple <# … #> Comments like the following situation, then everything between the first <# and the last #> was replaced by the single one external help link:

Function Set-FileContent-Encoded() {
	<#
	.SYNOPSIS
		…
	#>
	[CmdletBinding()]
	Param (
		…
	)

	Function ConvertHashtableTo-Object {
		<#
		.SYNOPSIS
			…
		#>
		…
	}
}

The Fix recognizes multiple Comment Based Help texts <# … #> correctly and replaces only the first one with the link to the external help
$Description can now contain Newlines / multiple Lines, they are properly indented.
$Description can now contain Newlines / multiple Lines, they are properly indented.
$Description can now contain Newlines / multiple Lines, they are properly indented.
@zloeber
zloeber merged commit a6cbb4f into ModuleBuild:master Mar 31, 2018
zloeber pushed a commit that referenced this pull request Apr 24, 2020
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