Skip to content

Commit

Permalink
Merge pull request #5636 from FabienTschanz/fix/csv-report
Browse files Browse the repository at this point in the history
Fix missing delimiter when called without parameter
  • Loading branch information
ykuijs authored Jan 17, 2025
2 parents 60c44d0 + fd31e88 commit 77b4059
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,9 @@
* Added support for the AllowGetOneLakeUDK, AllowMountDfCreation, AllowOneLakeUDK,
ArtifactOrgAppPreview properties.
* Fix values that have a zero length whitespace character.
* M365DSCReport
* Fix missing delimiter when called without the parameter.
FIXES [#5634](https://github.com/microsoft/Microsoft365DSC/issues/5634)
* M365DSCTelemetryEngine
* Report LCM details only if running as administrator.

Expand Down
1 change: 1 addition & 0 deletions Modules/Microsoft365DSC/Modules/M365DSCReport.psm1
Original file line number Diff line number Diff line change
Expand Up @@ -709,6 +709,7 @@ function New-M365DSCReportFromConfiguration
$delimiterParam = [System.Management.Automation.RuntimeDefinedParameter]::New("Delimiter", [System.String], $attributeCollection)
$delimiterParam.Value = ';' # default value, comma makes a mess when importing a CSV-file in Excel
$paramDictionary.Add("Delimiter", $delimiterParam)
$PSBoundParameters.Add("Delimiter", $delimiterParam.Value)
}
return $paramDictionary
}
Expand Down

0 comments on commit 77b4059

Please sign in to comment.