From 7e23670c8edde3eb46de57e3ee615d76f5141245 Mon Sep 17 00:00:00 2001 From: Flor Elisa Chacon Ochoa Date: Thu, 8 Aug 2024 11:44:29 -0700 Subject: [PATCH 1/5] Add PR trigger --- pipelines/winget-cli-restsource-ci-pr.yml | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/pipelines/winget-cli-restsource-ci-pr.yml b/pipelines/winget-cli-restsource-ci-pr.yml index de86f598..d3118678 100644 --- a/pipelines/winget-cli-restsource-ci-pr.yml +++ b/pipelines/winget-cli-restsource-ci-pr.yml @@ -1,12 +1,11 @@ # Copyright (c) Microsoft Corporation. All rights reserved # CI pipeline for winget-cli-restsource -# TODO: The forked pr builds need to have the same permission as non-forked ones to access -# project's internal nuget feed. Remove DisablePipelineConfigDetector variable when -# a long term fix is available. -variables: -- name: DisablePipelineConfigDetector - value: true +# PR triggers +pr: + branches: + include: + - master resources: repositories: From 1d569be2e9f3aa1ca27690747fd7917afd6284e5 Mon Sep 17 00:00:00 2001 From: Flor Elisa Chacon Ochoa Date: Thu, 8 Aug 2024 11:46:15 -0700 Subject: [PATCH 2/5] Wrong branch name... --- pipelines/winget-cli-restsource-ci-pr.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pipelines/winget-cli-restsource-ci-pr.yml b/pipelines/winget-cli-restsource-ci-pr.yml index d3118678..001d553f 100644 --- a/pipelines/winget-cli-restsource-ci-pr.yml +++ b/pipelines/winget-cli-restsource-ci-pr.yml @@ -5,7 +5,7 @@ pr: branches: include: - - master + - main resources: repositories: From 762cb23f714ad4e8927be19e176e72bcd6847bc7 Mon Sep 17 00:00:00 2001 From: Flor Elisa Chacon Ochoa Date: Fri, 9 Aug 2024 15:55:39 -0700 Subject: [PATCH 3/5] add paths --- pipelines/winget-cli-restsource-ci-pr.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/pipelines/winget-cli-restsource-ci-pr.yml b/pipelines/winget-cli-restsource-ci-pr.yml index 001d553f..8b29f34d 100644 --- a/pipelines/winget-cli-restsource-ci-pr.yml +++ b/pipelines/winget-cli-restsource-ci-pr.yml @@ -6,6 +6,10 @@ pr: branches: include: - main + paths: + include: + - pipelines/* + - src/* resources: repositories: From 7ef9582a99f89642e880520801a4bee1435d10a9 Mon Sep 17 00:00:00 2001 From: Flor Elisa Chacon Ochoa Date: Fri, 9 Aug 2024 16:12:38 -0700 Subject: [PATCH 4/5] Remove tag, paths --- pipelines/winget-cli-restsource-ci-pr.yml | 7 ------- 1 file changed, 7 deletions(-) diff --git a/pipelines/winget-cli-restsource-ci-pr.yml b/pipelines/winget-cli-restsource-ci-pr.yml index 8b29f34d..098935a2 100644 --- a/pipelines/winget-cli-restsource-ci-pr.yml +++ b/pipelines/winget-cli-restsource-ci-pr.yml @@ -6,10 +6,6 @@ pr: branches: include: - main - paths: - include: - - pipelines/* - - src/* resources: repositories: @@ -25,9 +21,6 @@ extends: name: Azure-Pipelines-1ESPT-ExDShared image: windows-2022 os: windows - customBuildTags: - # This is added by 1ES migration tool and it is okay to remove in the future. - - ES365AIMigrationTooling stages: - stage: Build_Publish_Test From 4a6519199538966d7a38628ec85d63a0fcd02cf7 Mon Sep 17 00:00:00 2001 From: Flor Elisa Chacon Ochoa Date: Fri, 9 Aug 2024 16:19:19 -0700 Subject: [PATCH 5/5] skip tagging --- pipelines/winget-cli-restsource-ci-pr.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/pipelines/winget-cli-restsource-ci-pr.yml b/pipelines/winget-cli-restsource-ci-pr.yml index 098935a2..76ede3cf 100644 --- a/pipelines/winget-cli-restsource-ci-pr.yml +++ b/pipelines/winget-cli-restsource-ci-pr.yml @@ -22,6 +22,10 @@ extends: image: windows-2022 os: windows + settings: + # Builds from forked repos do not have permissions to set tags + skipBuildTagsForGitHubPullRequests: true + stages: - stage: Build_Publish_Test jobs: