Skip to content

Conversation

@julien-lang
Copy link
Member

@julien-langjulien-lang commented Jun 2, 2025

This pull request updates the azure-pipelines/tests.yml file to improve readability by renaming matrix keys for Python versions.

BeforeAfter
imageimage
imageimage

Relates to shotgunsoftware/sg-jira-bridge#93

@julien-langjulien-lang requested a review from CopilotJune 2, 2025 20:11
Copy link
Contributor

CopilotAI 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

Updates the test pipeline template to use a customizable job display name and refactors the Python version matrix keys for readability

  • Added displayName using a job_name parameter
  • Renamed matrix entries from Python39/Python310/Python311 to Python 3.9/3.10/3.11
Comments suppressed due to low confidence (2)

azure-pipelines-templates/run-tests.yml:40

  • The template now references parameters.job_name, but it isn't defined in this file. Either add a job_name parameter or revert to using the existing name parameter to avoid broken references.
displayName: ${{parameters.job_name }} 

azure-pipelines-templates/run-tests.yml:48

  • Matrix keys with spaces and dots may generate invalid job identifiers in Azure Pipelines. Consider using identifier-friendly keys (e.g., Python39) and moving the human-readable name into a displayName setting.
Python 3.9: 

@julien-langjulien-lang changed the title TestBetter CI Job NameJun 2, 2025
@julien-langjulien-lang marked this pull request as ready for review June 2, 2025 21:51
@julien-langjulien-lang requested a review from CopilotJune 2, 2025 21:51
Copy link
Contributor

CopilotAI 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 improves the readability of the CI matrix by renaming Python version keys to include dots and spaces, and updates related comments.

  • Updated comment examples to show version names with spaces and dots
  • Renamed strategy matrix keys from Python39, Python310, Python311 to Python 3.9, Python 3.10, Python 3.11
  • Adjusted comments above the job definition to match the new naming format

@julien-langjulien-lang requested a review from a teamJune 2, 2025 21:54
@julien-langjulien-lang merged commit cfab2b4 into masterJun 3, 2025
22 checks passed
@julien-langjulien-lang deleted the julien/test-improve-ci branch June 3, 2025 14:08
eduardoChaucaGallegos pushed a commit that referenced this pull request Jul 15, 2025
Sign up for freeto 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

@julien-lang@carlos-villavicencio-adsk