Skip to content

XS✔ ◾ Fix PR Label Separators - #880

Merged
Muiris Woulfe (muiriswoulfe) merged 2 commits into
mainfrom
fix/878-hyphen-pr-labels
Oct 23, 2026
Merged

Muiris Woulfe (muiriswoulfe) merged 2 commits into
mainfrom
fix/878-hyphen-pr-labels

Conversation

@muiriswoulfe

Copy link
Copy Markdown
Member

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 tests
  • Manual tests

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.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: c994e3a0-4df2-45e4-a83a-6d00a15e8cf8
@github-actions

github-actions Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

PR Metrics

✔ Thanks for keeping your pull request small.
✔ Thanks for adding tests.

Lines
Product Code 6
Test Code 196
Subtotal 202
Ignored Code 28
Total 230

Metrics computed by PR Metrics. Add it to your Azure DevOps and GitHub PRs!

@github-actions github-actions Bot changed the title Fix PR Label Separators XS✔ ◾ Fix PR Label Separators Oct 23, 2026
Comment thread dist/index.mjs
Comment thread dist/resources.resjson
Comment thread package-lock.json

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.

🟢 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

@github-actions

github-actions Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Super-linter summary

Language Validation result
CHECKOV Pass ✅
EDITORCONFIG Pass ✅
GITHUB_ACTIONS Pass ✅
GITHUB_ACTIONS_ZIZMOR Pass ✅
GITLEAKS Pass ✅
GIT_MERGE_CONFLICT_MARKERS Pass ✅
JSCPD Pass ✅
JSON_PRETTIER Pass ✅
MARKDOWN Pass ✅
MARKDOWN_PRETTIER Pass ✅
NATURAL_LANGUAGE Pass ✅
POWERSHELL Pass ✅
PRE_COMMIT Pass ✅
SPELL_CODESPELL Pass ✅
TRIVY Pass ✅
TYPESCRIPT_PRETTIER Pass ✅
XML Pass ✅
YAML Pass ✅
YAML_PRETTIER Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter
Super-linter revision: 2da136927bd4a73596db63044b504547c62cb854
Super-linter version: 9.0.0

PR-Metrics-Automation-Run: 39950031885
PR-Metrics-Automation-Attempt: 1
Copilot AI balanced review requested due to automatic review settings October 23, 2026 12:00

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.

🟢 Approval recommended

The focused separator change is consistent with its regression coverage, documentation, and release metadata, with no unresolved blocking findings.

No open issues 🎉

Review effort: Balanced

@muiriswoulfe
Muiris Woulfe (muiriswoulfe) merged commit 050a64a into main Oct 23, 2026
37 checks passed
@muiriswoulfe
Muiris Woulfe (muiriswoulfe) deleted the fix/878-hyphen-pr-labels branch October 23, 2026 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Removing a stale pr-metrics:<size> label fails with HTTP 400 "potentially dangerous Request.Path value (:)" (v1.8.1, Azure Pipelines)

3 participants