Skip to content

Conversation

@julien-lang
Copy link
Member

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

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

No longer use a matrix strategy because only work with one dimension
But we have two dimension: OS and Python version
@julien-lang julien-lang requested a review from Copilot December 12, 2025 18:28
Copy link

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 simplifies the Azure Pipeline CI configuration by moving from a matrix strategy to explicit parameter-based job generation. The changes enable testing across both OS and Python version dimensions using nested loops instead of the previous single-dimension matrix approach.

Key Changes:

  • Refactored pipeline templates to use explicit parameters instead of matrix strategy
  • Added nested loops to generate jobs for each OS and Python version combination
  • Standardized task definitions using Bash@3 instead of inline scripts

Reviewed changes

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

File Description
azure-pipelines/tests.yml Converted from matrix-based to parameter-based job template with explicit python_version parameter
azure-pipelines.yml Added parameter definitions for OS and Python versions with nested loops to generate test jobs

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

@julien-lang julien-lang requested a review from a team December 16, 2025 20:09
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.

2 participants