Skip to content

SG-40980 Simplify Azure Pipeline CI pipeline#436

Merged
julien-lang merged 5 commits into
masterfrom
ticket/SG-40980-simplify-ci-pipeline
Jan 5, 2026
Merged

SG-40980 Simplify Azure Pipeline CI pipeline#436
julien-lang merged 5 commits into
masterfrom
ticket/SG-40980-simplify-ci-pipeline

Conversation

@julien-lang

@julien-lang julien-lang commented Dec 12, 2025

Copy link
Copy Markdown
Member

Simplify the os_version/python_version combination for calling the test template.

No longer use a matrix strategy because only work with one dimension.
Define and use parameter for each version dimension and use the each operator for looping over both parameters.

Replicated changes on multiple repositories

@julien-lang julien-lang changed the title Tests SG-40980 Simplify Azure Pipeline CI pipeline Dec 12, 2025
@julien-lang julien-lang requested a review from Copilot December 12, 2025 18:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 simplifies the Azure Pipelines CI configuration by introducing parameterized loops to reduce code duplication. Instead of explicitly defining separate job templates for each OS, the pipeline now uses parameters to dynamically generate jobs for all combinations of operating systems and Python versions.

Key Changes:

  • Introduced python_versions and os_versions parameters to define test matrix configurations
  • Replaced explicit job template calls with nested loops that generate jobs dynamically
  • Updated the run-tests.yml template to accept Python version as a parameter and removed the strategy matrix

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
azure-pipelines.yml Added parameters for Python and OS versions, replaced explicit job templates with nested loops to generate test jobs dynamically
azure-pipelines-templates/run-tests.yml Added python_version parameter, removed strategy matrix, converted OS name comparison to vm_image prefix checks, standardized task syntax to Bash@3

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread azure-pipelines-templates/run-tests.yml Outdated
Comment thread azure-pipelines-templates/run-tests.yml Outdated
Comment thread azure-pipelines-templates/run-tests.yml
@julien-lang julien-lang requested a review from a team December 16, 2025 20:09
@julien-lang julien-lang merged commit 13be391 into master Jan 5, 2026
22 checks passed
@julien-lang julien-lang deleted the ticket/SG-40980-simplify-ci-pipeline branch January 5, 2026 19:58
@julien-lang julien-lang restored the ticket/SG-40980-simplify-ci-pipeline branch January 5, 2026 21:43
@julien-lang julien-lang deleted the ticket/SG-40980-simplify-ci-pipeline branch January 5, 2026 21:43
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.

3 participants