Skip to content

Conversation

@YuliiaKovalova
Copy link
Member

Fixes:

  • Changed the templates from having stages: at the top level to jobs:. This is because it's being included as a job template, not a stage template.
  • Post-build Template Location: Moved the post-build template from inside the job template (where it doesn't belong) to the main pipelines at the stage level where it should be.

Copilot AI review requested due to automatic review settings August 25, 2025 11:30
@YuliiaKovalova YuliiaKovalova requested a review from a team as a code owner August 25, 2025 11:30
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors Azure DevOps pipeline configuration by moving the post-build template from a job-level template to the main pipeline stage level, where it belongs according to Azure DevOps pipeline structure conventions.

  • Moved post-build template from job template to main pipeline files at stage level
  • Removed post-build template and related parameters from the job template
  • Added post-build template to both main pipeline and experimental performance pipeline

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
azure-pipelines/.vsts-dotnet-exp-perf.yml Added post-build template at stage level for experimental performance pipeline
azure-pipelines/.vsts-dotnet-build-jobs.yml Removed post-build template and enableSigningValidation parameter from job template
.vsts-dotnet.yml Added post-build template at stage level for main pipeline and removed conditional variable

@YuliiaKovalova YuliiaKovalova enabled auto-merge (squash) August 25, 2025 12:23
@YuliiaKovalova YuliiaKovalova merged commit e67b64e into main Aug 25, 2025
9 checks passed
@YuliiaKovalova YuliiaKovalova deleted the dev/ykovalova/more_yml_changes branch August 25, 2025 12:40
Copilot AI added a commit that referenced this pull request Aug 25, 2025
JanProvaznik pushed a commit that referenced this pull request Aug 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants