From 45354d46c41faea4bf26f3d9a3905f63da6aac71 Mon Sep 17 00:00:00 2001 From: Kyriakos Sidiropoulos Date: Tue, 5 Nov 2024 12:25:47 +0100 Subject: [PATCH] clean (Laerdal.McuMgr.slnx): delete azure-pipelines.yml since it is no longer being needed [skip ci] --- Laerdal.McuMgr.slnx | 41 +++++----- Laerdal.McuMgr.slnx.DotSettings.user | 10 ++- azure-pipelines.yml | 116 --------------------------- 3 files changed, 28 insertions(+), 139 deletions(-) delete mode 100644 azure-pipelines.yml diff --git a/Laerdal.McuMgr.slnx b/Laerdal.McuMgr.slnx index c366a772..6bcb357f 100644 --- a/Laerdal.McuMgr.slnx +++ b/Laerdal.McuMgr.slnx @@ -1,23 +1,22 @@  - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + \ No newline at end of file diff --git a/Laerdal.McuMgr.slnx.DotSettings.user b/Laerdal.McuMgr.slnx.DotSettings.user index f9fbd1a2..22a122b2 100644 --- a/Laerdal.McuMgr.slnx.DotSettings.user +++ b/Laerdal.McuMgr.slnx.DotSettings.user @@ -1,8 +1,11 @@  + ForceIncluded - <SessionState ContinuousTestingMode="0" IsActive="True" Name="All tests from Solution" xmlns="urn:schemas-jetbrains-com:jetbrains-ut-session"> - <Solution /> + <SessionState ContinuousTestingMode="0" IsActive="True" Name="All tests from Solution" xmlns="urn:schemas-jetbrains-com:jetbrains-ut-session"> + <Solution /> </SessionState> + + True True @@ -37,6 +40,9 @@ True True True + + + True True True diff --git a/azure-pipelines.yml b/azure-pipelines.yml deleted file mode 100644 index e936613b..00000000 --- a/azure-pipelines.yml +++ /dev/null @@ -1,116 +0,0 @@ -# to future maintainers considering that this is an open-source project please be mindful of the fact that this file is PUBLIC -# to future maintainers so any sensitive pieces of info (tokens, passwords, etc) should be stored in ENCRYPTED Azure DevOps pipeline variables -# to future maintainers -# to future maintainers in the same vein of thought its important to always keep an eye on the pipelines trigger-settings for PR validation -# to future maintainers to ensure that PR builds coming from people outside Laerdal fire if and only a Laerdal dev has explicitly comments -# to future maintainers on the PR with the following string [build] if this string is not present the PR build should NOT fire - -trigger: - branches: - include: - - '*' - -pool: - name: 'Azure Pipelines' - vmImage: 'macOS-13' # to future maintainers if you ever want to switch back to azure be warned that you will need to first make sure that macos14 is available on azure if macos14 - # to future maintainers is not available on azure you will run into trouble making the build work for the latest versions of ios sdks (17+) and maccatalyst sdks - -variables: - - group: shared-variables # needed for the github access token which is used to create new releases in github - - - name: Laerdal_Test_Results_Folderpath - value: '$(Build.Repository.LocalPath)/TestResults' - - - name: is_from_fork # its vital to use the notation variables['x'] for the environment variables below if we use variables.x the value will be evaluated as empty! - value: ${{ eq(variables['System.PullRequest.IsFork'], 'True') }} - - - name: has_green_build_reason - value: ${{ in(variables['Build.Reason'], 'PullRequest', 'Manual', 'Schedule', 'BuildCompletion', 'ResourceTrigger') }} - - - name: is_triggered_from_central_branches - value: ${{ in(variables['Build.SourceBranch'], 'refs/heads/main', 'refs/heads/master', 'refs/heads/develop') }} - - - name: contains_build_keyword_in_commit_message - value: ${{ contains(variables['Build.SourceVersionMessage'], '[build]') }} - - - name: should_build - ${{ if eq(variables.is_from_fork, True) }}: # order must be checked first - value: 'No [is from fork]' # we want to explicitly disable any and all activity related to 3rd party forks in github - ${{ elseif eq(variables.has_green_build_reason, True) }}: - value: 'Yes [has green build reason]' - ${{ elseif eq(variables.is_triggered_from_central_branches, True) }}: - value: 'Yes [is triggered from central branches]' - ${{ elseif eq(variables.contains_build_keyword_in_commit_message, True) }}: - value: 'Yes [contains build keyword in commit message]' - ${{ else }}: - value: 'No [humdrum feature branch commit]' - -resources: - repositories: - - repository: self - type: git - ref: refs/heads/develop - -jobs: - - - job: 'Skip_Build' - condition: ${{ startsWith(variables.should_build, 'No ') }} - displayName: "Should we auto-build? ... ${{ variables.should_build }} (is_from_fork=__${{ variables.is_from_fork }}__ ** has_green_build_reason=__${{ variables.has_green_build_reason }}__ ** is_triggered_from_central_branches=__${{ variables.is_triggered_from_central_branches }}__ ** contains_build_keyword_in_commit_message=__${{ variables.contains_build_keyword_in_commit_message }}__ ** IsFork=__${{ variables['System.PullRequest.IsFork'] }}__ ** Build.Reason=__${{ variables['Build.Reason'] }}__ ** SourceBranch=__${{ variables['Build.SourceBranch'] }}__ ** SourceVersionMessage=__${{ variables['Build.SourceVersionMessage'] }}__)" - steps: - - checkout: none # keep this otherwise it will make a full checkout of the repo unnecessarily - displayName: '(Skipping checkout completely ...)' - - script: 'echo "##vso[task.complete result=SucceededWithIssues;]Build skipped because this is a PR coming from a fork or because this is from a feature branch in which the latest commit message doesnt contain the keyword [build]"' - displayName: '(Set Warning-Icon ... )' - - - job: 'Build' - condition: ${{ startsWith(variables.should_build, 'Yes ') }} - displayName: 'Build Agent' - steps: - - - checkout: self - displayName: '🔽 Checkout' - clean: true - fetchTags: false - fetchDepth: 0 - persistCredentials: True - - - task: Bash@3 - displayName: '🛠 Setup Build Environment' - inputs: - targetType: filePath - filePath: 'Laerdal.Scripts/Laerdal.SetupBuildEnvironment.sh' - arguments: ' "https://pkgs.dev.azure.com/LaerdalMedical/_packaging/LaerdalNuGet/nuget/v3/index.json" "$(Laerdal.NugetFeed.Username)" "$(Laerdal.NugetFeed.AccessToken)" "$(Build.ArtifactStagingDirectory)/Artifacts/" ' - - # notice that explicitly disabling parallelization via m:1 below is vital because if parallelization is enabled the ios and android - # builds will fail with cryptic errors due to multiple target-frameworks are involved coupled with the custom build logic for native libs - # - # also noticed that we intentionally skip the maccatalyst build on azure because under net8 we need macos14 to be available on azure but - # it is not available just yet in 2024 Q1 :( - - task: DotNetCoreCLI@2 - displayName: '🏗 📦 Build, Pack & Announce New Release (if appropriate)' - inputs: - command: 'custom' - custom: 'msbuild' - arguments: 'Laerdal.Scripts/Laerdal.Builder.targets -m:1 -p:Should_Skip_MacCatalyst="true" -p:Laerdal_Gradle_Path="/usr/local/opt/gradle@7/bin/gradle" -p:Laerdal_Test_Results_Folderpath="$(Laerdal_Test_Results_Folderpath)" -p:Laerdal_Github_Access_Token="$(Github.ComponentsTeam.AccessToken)" -p:Laerdal_Repository_Path="$(Repository.Path)" -p:Laerdal_Source_Branch="$(Build.SourceBranch)" -p:PackageOutputPath="$(Build.ArtifactStagingDirectory)/Artifacts/" ' - - - task: PublishTestResults@2 - displayName: '📡 Publish Test Results' - inputs: - testRunTitle: 'McuMgr Test Suite (Platform-Agnostic)' - mergeTestResults: true - testResultsFiles: | - $(Laerdal_Test_Results_Folderpath)/**/TEST-*.xml - $(Laerdal_Test_Results_Folderpath)/**/TEST-*.trx - testResultsFormat: 'XUnit' - publishRunAttachments: true - failTaskOnFailedTests: true - failTaskOnMissingResultsFile: true - - # bare in mind that this task doesnt support wildcards - - task: PublishBuildArtifacts@1 - displayName: '📡 Publish Artifacts' - inputs: - ArtifactName: 'drop' - PathtoPublish: '$(Build.ArtifactStagingDirectory)/Artifacts/' - publishLocation: 'Container' -