Remove deprecated GitVersion output and variable formats

Dropped support for `GitVersion.<property>` format in outputs and variables to simplify the codebase and improve clarity. Tests were updated to reflect these changes, ensuring compatibility with the remaining formats.
arturcic · Dec 10, 2024 · 0dbd9f6 · 0dbd9f6
1 parent bc8cf5c
commit 0dbd9f6
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 17 deletions.
diff --git a/src/__tests__/tools/gitversion/runner.spec.ts b/src/__tests__/tools/gitversion/runner.spec.ts
@@ -75,9 +75,6 @@ describe('GitVersion Runner', () => {
             expect(getEnv('GitVersion_Major')).toBeDefined()
             expect(getEnv('GitVersion_Minor')).toBeDefined()
             expect(getEnv('GitVersion_Patch')).toBeDefined()
-            expect(getEnv('GitVersion.Major')).toBeDefined()
-            expect(getEnv('GitVersion.Minor')).toBeDefined()
-            expect(getEnv('GitVersion.Patch')).toBeDefined()
 
             expect(getEnv('major')).toBeDefined()
             expect(getEnv('minor')).toBeDefined()

diff --git a/src/__tests__/tools/gitversion/tool.spec.ts b/src/__tests__/tools/gitversion/tool.spec.ts
@@ -86,12 +86,6 @@ describe('GitVersionTool', () => {
             expect(outputs.get('GitVersion_SemVer')).toBe('1.2.3-alpha.1')
             expect(outputs.get('GitVersion_FullSemVer')).toBe('1.2.3-alpha.1')
 
-            expect(outputs.get('GitVersion.Major')).toBe('1')
-            expect(outputs.get('GitVersion.Minor')).toBe('2')
-            expect(outputs.get('GitVersion.Patch')).toBe('3')
-            expect(outputs.get('GitVersion.SemVer')).toBe('1.2.3-alpha.1')
-            expect(outputs.get('GitVersion.FullSemVer')).toBe('1.2.3-alpha.1')
-
             expect(variables.get('major')).toBe('1')
             expect(variables.get('minor')).toBe('2')
             expect(variables.get('patch')).toBe('3')
@@ -103,12 +97,6 @@ describe('GitVersionTool', () => {
             expect(variables.get('GitVersion_Patch')).toBe('3')
             expect(variables.get('GitVersion_SemVer')).toBe('1.2.3-alpha.1')
             expect(variables.get('GitVersion_FullSemVer')).toBe('1.2.3-alpha.1')
-
-            expect(variables.get('GitVersion.Major')).toBe('1')
-            expect(variables.get('GitVersion.Minor')).toBe('2')
-            expect(variables.get('GitVersion.Patch')).toBe('3')
-            expect(variables.get('GitVersion.SemVer')).toBe('1.2.3-alpha.1')
-            expect(variables.get('GitVersion.FullSemVer')).toBe('1.2.3-alpha.1')
         })
     })
 

diff --git a/src/tools/gitversion/tool.ts b/src/tools/gitversion/tool.ts
@@ -58,10 +58,8 @@ export class GitVersionTool extends DotnetTool {
                 }
                 this.buildAgent.setOutput(name, value)
                 this.buildAgent.setOutput(`GitVersion_${property}`, value)
-                this.buildAgent.setOutput(`GitVersion.${property}`, value)
                 this.buildAgent.setVariable(name, value)
                 this.buildAgent.setVariable(`GitVersion_${property}`, value)
-                this.buildAgent.setVariable(`GitVersion.${property}`, value)
             } catch (_error) {
                 this.buildAgent.error(`Unable to set output/variable for ${property}`)
             }