Skip to content

Commit

Permalink
fix (Laerdal.Builder.targets): fix a bug which was causing the build …
Browse files Browse the repository at this point in the history
…to not tag git when it would be invoked from develop
  • Loading branch information
ksidirop-laerdal committed Nov 12, 2024
1 parent e60baad commit 8ca3c04
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions Laerdal.Scripts/Laerdal.Builder.targets
Original file line number Diff line number Diff line change
Expand Up @@ -24,17 +24,17 @@
<PackageOutputPath Condition=" '$(PackageOutputPath)' == '' ">$(BUILD_ARTIFACTSTAGINGDIRECTORY)</PackageOutputPath>
<PackageOutputPath Condition=" '$(PackageOutputPath)' == '' ">$([System.IO.Path]::Combine($(MSBuildThisFileDirectory), `..`, `Artifacts/`))</PackageOutputPath>

<Laerdal_Source_Branch Condition=" '$(Laerdal_Source_Branch)' == '' ">$(BUILD_SOURCEBRANCH)</Laerdal_Source_Branch>
<Laerdal_Repository_Path Condition=" '$(Laerdal_Repository_Path)' == '' ">$(BUILD_REPOSITORY_NAME)</Laerdal_Repository_Path>
<Laerdal_Dependency_Tracker_Server_Url Condition=" '$(Laerdal_Dependency_Tracker_Server_Url)' == '' ">https://dep-tracker.laerdal.com/api/api/v1/bom</Laerdal_Dependency_Tracker_Server_Url>

<!-- https://docs.gitlab.com/ee/ci/variables/predefined_variables.html -->
<!-- https://help.github.com/en/actions/configuring-and-managing-workflows/using-environment-variables -->
<!-- https://learn.microsoft.com/en-us/azure/devops/pipelines/build/variables?view=azure-devops&tabs=yaml&WT.mc_id=DT-MVP-5003978#system-variables -->
<Is_CI_Build Condition=" '$(TF_BUILD)' == 'true' or '$(GITHUB_ACTIONS)' == 'true' or '$(GITLAB_CI)' == 'true' ">true</Is_CI_Build>
<Is_Core_Branch Condition=" '$(Laerdal_Source_Branch)' == 'refs/heads/main' or '$(Laerdal_Source_Branch)' == 'refs/heads/master' or '$(Laerdal_Source_Branch)' == 'refs/heads/develop' ">true</Is_Core_Branch>
<Is_Pull_Request Condition=" '$(Laerdal_Source_Branch.StartsWith(refs/pull))' == 'true' and '$(Laerdal_Source_Branch.EndsWith(/merge))' == 'true' ">true</Is_Pull_Request>

<Laerdal_Dependency_Tracker_Server_Url Condition=" '$(Laerdal_Dependency_Tracker_Server_Url)' == '' ">https://dep-tracker.laerdal.com/api/api/v1/bom</Laerdal_Dependency_Tracker_Server_Url>

<Laerdal_Source_Branch Condition=" '$(Laerdal_Source_Branch)' == '' ">$(BUILD_SOURCEBRANCH)</Laerdal_Source_Branch>
<Laerdal_Repository_Path Condition=" '$(Laerdal_Repository_Path)' == '' ">$(BUILD_REPOSITORY_NAME)</Laerdal_Repository_Path>
<Laerdal_Should_Tag_And_Release Condition=" '$(Laerdal_Should_Tag_And_Release)' == '' and '$(Is_Core_Branch)' == 'true' ">True</Laerdal_Should_Tag_And_Release>
<Laerdal_Should_Generate_and_Upload_Sbom Condition=" '$(Laerdal_Should_Generate_and_Upload_Sbom)' == '' and ( '$(Is_Core_Branch)' == 'true' or '$(Is_Pull_Request)' == 'true' ) ">True</Laerdal_Should_Generate_and_Upload_Sbom>

Expand Down

0 comments on commit 8ca3c04

Please sign in to comment.