Remove link verification from ci job#47756
Conversation
There was a problem hiding this comment.
Pull request overview
This PR removes the verify-links.yml template invocation from the Analyze job in eng/pipelines/templates/jobs/ci.yml, so the Azure Pipelines CI no longer runs markdown link verification. Link verification for pull requests is already provided by the parallel GitHub Actions workflow .github/workflows/verify-links.yml, which runs the same get-markdown-files-from-changed-files.ps1 and Verify-Links.ps1 scripts on pull_request events, so this removes a redundant check from the Azure Pipelines path.
Changes:
- Removed the
verify-links.ymlstep (and itsBuild.Reason-conditional parameters) from theAnalyzejob in the CI pipeline.
The removal is clean: the step previously sat between check-spelling.yml and the DownloadPipelineArtifact@2 task, and the surrounding YAML remains valid. One behavioral note worth a human's confirmation: the removed step also ran link verification on sdk/<ServiceDirectory> for non-PR (scheduled/manual) builds, whereas the GitHub Actions workflow only triggers on pull_request/check_run events — so link verification for non-PR builds is no longer performed. This appears intentional given the PR's stated purpose, but is a CI/CD coverage tradeoff.
This pull request removes the link verification step from the CI pipeline configuration. The
verify-links.ymltemplate, which checked for valid links in markdown files, will no longer run as part of the pipeline.Pipeline configuration changes:
verify-links.ymltemplate step from the CI pipeline, eliminating automated link verification for markdown files during CI runs.