Repository navigation
XS✔ ◾ Fix PR Label Separators - #880
Merged
Merged
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: c994e3a0-4df2-45e4-a83a-6d00a15e8cf8
Muiris Woulfe (muiriswoulfe)
requested review from
a team
and
a balanced review from Copilot
October 23, 2026 11:56
Contributor
PR Metrics✔ Thanks for keeping your pull request small.
Metrics computed by PR Metrics. Add it to your Azure DevOps and GitHub PRs! |
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The focused separator change, legacy-label preservation, regression coverage, and release metadata are consistent, with no blocking issues identified.
No open issues 🎉
What changed in this PR
Updates PR Metrics labels to use hyphens, avoiding Azure Repos deletion-path errors reported in #878 while preserving existing colon labels.
Changes:
- Switches managed size and test-status labels to hyphen-separated names.
- Updates tests, including regressions for size changes, repeated runs, and legacy-label preservation.
- Updates documentation and release metadata for version 1.8.2.
| File | Description |
|---|---|
| src/vss-extension.json | Bumps extension version to 1.8.2. |
| src/task/tests/wrappers/octokitWrapper.labels.spec.ts | Updates label names and expected API requests. |
| src/task/tests/repos/reposInvoker.spec.ts | Updates label-routing expectations. |
| src/task/tests/repos/gitHubReposInvoker.labels.spec.ts | Updates GitHub label test fixtures. |
| src/task/tests/repos/azureReposInvoker.labels.spec.ts | Updates Azure Repos label test fixtures. |
| src/task/tests/pullRequests/pullRequestLabels.spec.ts | Adds separator and legacy-preservation regressions. |
| src/task/tests/metrics/codeMetricsCalculator.labels.spec.ts | Updates integration expectations for hyphen labels. |
| src/task/task.loc.json | Bumps localized task version. |
| src/task/task.json | Updates task version and display name. |
| src/task/Strings/resources.resjson/en-US/resources.resjson | Updates localized display name. |
| src/task/src/utilities/constants.ts | Updates the API user-agent version. |
| src/task/src/pullRequests/pullRequestLabels.ts | Creates and reconciles hyphen-separated labels. |
| README.md | Updates label documentation and version examples. |
| package.json | Bumps package version to 1.8.2. |
| package-lock.json | Synchronizes package version metadata. |
| docs/azure-pipelines-task.md | Documents hyphen-separated labels. |
| dist/resources.resjson | Updates the packaged display name. |
| .github/release-version.json | Advances the next release version to 1.8.3. |
| .github/release-publish-trigger.txt | Sets the release trigger to 1.8.2. |
Review effort: Balanced
Contributor
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
PR-Metrics-Automation-Run: 39950031885 PR-Metrics-Automation-Attempt: 1
Keith Cully (K-Cully)
approved these changes
Oct 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #878.
Use hyphens in size and test-status labels so Azure Repos can remove stale labels without path errors. Preserve existing colon labels and prepare version 1.8.2.
Testing
Test Types
Unit Test Coverage
npm run test:no-install: passed, 904 tests. Statements, branches, functions, and lines: 100%.Other Test Details
Regression tests cover size changes, repeated runs, and preservation of existing colon labels. Both distribution builds and focused linting passed. Manual tests were not run.
Screenshots
Not applicable.