-
Notifications
You must be signed in to change notification settings - Fork 206
Better CI Job Name #388
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Better CI Job Name #388
Conversation
There was a problem hiding this 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 ajob_name
parameter - Renamed matrix entries from
Python39
/Python310
/Python311
toPython 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 ajob_name
parameter or revert to using the existingname
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 adisplayName
setting.
Python 3.9:
There was a problem hiding this 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
toPython 3.9
,Python 3.10
,Python 3.11
- Adjusted comments above the job definition to match the new naming format
This pull request updates the
azure-pipelines/tests.yml
file to improve readability by renaming matrix keys for Python versions.Relates to shotgunsoftware/sg-jira-bridge#93