Skip to content

Commit

Permalink
Revert "[CI] Create the initial stage to start running APIScan. (#19828
Browse files Browse the repository at this point in the history
…)" (#20056)

This reverts commit b7f4bef.
  • Loading branch information
rolfbjarne committed Feb 7, 2024
1 parent 5cd497a commit 02947b2
Show file tree
Hide file tree
Showing 8 changed files with 20 additions and 189 deletions.
44 changes: 0 additions & 44 deletions tools/devops/automation/scripts/Governance.psm1

This file was deleted.

1 change: 0 additions & 1 deletion tools/devops/automation/scripts/MaciosCI.psd1
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,6 @@ ScriptsToProcess = @(
NestedModules = @(
'Artifacts.psm1',
'GitHub.psm1',
'Governance.psm1',
'MLaunch.psm1',
'RemoteMac.psm1',
'StaticPages.psm1',
Expand Down
13 changes: 1 addition & 12 deletions tools/devops/automation/templates/common/configure.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,22 +80,11 @@ steps:
$testMatrix = $testMatrix | ConvertFrom-Json | ConvertTo-Json -Compress
Write-Host "##vso[task.setvariable variable=TEST_MATRIX;isOutput=true]$testMatrix"
name: test_matrix
displayName: 'Create tests strategy matrix'
displayName: 'Create strategy matrix'
env:
TEST_CONFIGURATIONS: ${{ convertToJson(parameters.testConfigurations) }}
SUPPORTED_PLATFORMS: ${{ convertToJson(parameters.supportedPlatforms) }}

- pwsh: |
Import-Module $Env:SYSTEM_DEFAULTWORKINGDIRECTORY/xamarin-macios/tools/devops/automation/scripts/MaciosCI.psd1
gci env: | sort | format-table -autosize -wrap
$apiScanMatrix = Get-APIScanConfiguration -EnabledPlatforms "$Env:CONFIGURE_PLATFORMS_DOTNET_PLATFORMS"
# compress the json to remove any newlines, because we can't set the variable below if the json has any newlines
Write-Host "$apiScanMatrix"
$apiScanMatrix = $apiScanMatrix | ConvertFrom-Json | ConvertTo-Json -Compress
Write-Host "##vso[task.setvariable variable=APISCAN_MATRIX;isOutput=true]$apiScanMatrix"
name: apiscan_matrix
displayName: 'Create APIScan matrix'

# upload config to be consumed later
- ${{ if eq(parameters.uploadArtifacts, true) }}:
- task: PublishPipelineArtifact@1
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ parameters:

steps:

- template: ../common/checkout.yml
- template: ./common/checkout.yml
parameters:
isPR: ${{ parameters.isPR }}
repositoryAlias: ${{ parameters.repositoryAlias }}
Expand All @@ -31,6 +31,7 @@ steps:
Dir $(System.DefaultWorkingDirectory)
displayName: Show directories


- powershell: |
Get-ChildItem -Recurse -Path $(Build.SourcesDirectory)/maccore
displayName: Show maccore content
Expand Down
28 changes: 0 additions & 28 deletions tools/devops/automation/templates/governance/apiscan.yml

This file was deleted.

69 changes: 0 additions & 69 deletions tools/devops/automation/templates/governance/stage.yml

This file was deleted.

28 changes: 0 additions & 28 deletions tools/devops/automation/templates/governance/tsa-upload.yml

This file was deleted.

23 changes: 17 additions & 6 deletions tools/devops/automation/templates/main-stage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -277,12 +277,23 @@ parameters:
stages:

- ${{ if eq(parameters.runGovernanceTests, true) }}:
- template: ./governance/stage.yml
parameters:
isPR: ${{ parameters.isPR }}
repositoryAlias: ${{ parameters.repositoryAlias }}
commit: ${{ parameters.commit }}
stageDisplayNamePrefix: ${{ parameters.stageDisplayNamePrefix }}
- stage: governance_checks
displayName: '${{ parameters.stageDisplayNamePrefix }}Governance Checks'
dependsOn: build_packages
${{ if and(ne(parameters.dependsOn, ''), ne(parameters.dependsOnResult, '')) }}:
condition: eq(dependencies.${{ parameters.dependsOn }}.result, '${{ parameters.dependsOnResult }}')
jobs:
- job: governance
displayName: 'Governance Checks'
pool:
vmImage: windows-latest
steps:
- template: governance-checks.yml
parameters:
isPR: ${{ parameters.isPR }}
repositoryAlias: ${{ parameters.repositoryAlias }}
commit: ${{ parameters.commit }}


- ${{ if parameters.isPR }}:
- stage: clean
Expand Down

8 comments on commit 02947b2

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ API diff for current PR / commit

Legacy Xamarin (No breaking changes)
  • iOS (no change detected)
  • tvOS (no change detected)
  • watchOS (no change detected)
  • macOS (no change detected)
NET (empty diffs)
  • iOS: (empty diff detected)
  • tvOS: (empty diff detected)
  • MacCatalyst: (empty diff detected)
  • macOS: (empty diff detected)

✅ API diff vs stable

Legacy Xamarin (No breaking changes)
.NET (No breaking changes)
Legacy Xamarin (stable) vs .NET

ℹ️ Generator diff

Generator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes)

Pipeline on Agent
Hash: 02947b2a6d02f7273d2b816dd815d5779fcaf8cb [CI build]

@vs-mobiletools-engineering-service2

This comment was marked as outdated.

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📚 [CI Build] Artifacts 📚

Packages generated

View packages

Pipeline on Agent
Hash: [CI build]

Please sign in to comment.