Fixed bugs in 'DscTagging' (#216)

dsccommunity · Jul 27, 2023 · a6d8b9f · a6d8b9f
1 parent 13d1e6b
commit a6d8b9f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -38,6 +38,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - `WindowsOptionalFeatures` and `WindowsFeatures` are using the DSC resource in `xPSDesiredStateConfiguration` now.
 - `CertificateRequests` supports multiple certificates with the same issuer and subject by making friendlyName a mandatory (key) parameter.
 
+### Fixed
+
+- Fixed bugs in 'DscTagging'
+
 ## [0.9.0] - 2023-02-08
 
 ### Added

diff --git a/source/DSCResources/DscTagging/DscTagging.schema.psm1 b/source/DSCResources/DscTagging/DscTagging.schema.psm1
@@ -28,7 +28,7 @@ configuration DscTagging {
     Import-DscResource -ModuleName PSDesiredStateConfiguration
     Import-DscResource -ModuleName xPSDesiredStateConfiguration
 
-    if ($DisableGitCommitId -ne $false)
+    if (-not $DisableGitCommitId)
     {
         $gitCommitId = git log -n 1 *>&1
         $gitCommitId = if ($gitCommitId -like '*fatal*')
@@ -85,7 +85,7 @@ configuration DscTagging {
     {
         Key       = 'HKEY_LOCAL_MACHINE\SOFTWARE\DscTagging'
         ValueName = 'BuildNumber'
-        ValueData = "$($env:BHBuildNumber)"
+        ValueData = "$($env:BUILD_BUILDID)"
         ValueType = 'String'
         Ensure    = 'Present'
         Force     = $true