Skip to content

Add description and polish pipeline Run dialog#645

Merged
karianna merged 3 commits into
microsoft/mainfrom
dev/dagood/pipe-clarity
Jul 8, 2026
Merged

Add description and polish pipeline Run dialog#645
karianna merged 3 commits into
microsoft/mainfrom
dev/dagood/pipe-clarity

Conversation

@dagood

@dagood dagood commented Jul 7, 2026

Copy link
Copy Markdown
Member

A few changes make the pipeline Run dialogs more usable for devs without full context:

Add an explicit description of what the pipeline is for by including a parameter with only one choice. The radio select is a bit ugly, but some emoji make it blend in a little, and the pipeline filename seems like a reasonable thing to include that makes the line seem less useless/out-of-place:

image

(This change will also be done in go-infra-images, and that change is really the one that directly addresses the recommendation in the RCA, by making it clear that that pipeline is the wrong one to use for this.)

Also, clarify the parameter display names.

Copilot AI review requested due to automatic review settings July 7, 2026 16:49
@dagood dagood requested a review from a team as a code owner July 7, 2026 16:49
@dagood dagood changed the title Add description to pipeline Run dialog Add description and polish pipeline Run dialog Jul 7, 2026

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.

Pull request overview

Improves Azure DevOps “Run pipeline” UX for this repo’s pipelines by making the run dialog more self-explanatory and adding easy-to-find links back to the corresponding pipeline definitions.

Changes:

  • Add an _info runtime parameter (single-choice string) to surface a brief “what this pipeline is for” description in the Run dialog.
  • Add “official/unofficial/public” pipeline definition links near the top of relevant pipeline YAMLs.
  • Polish several runtime parameter displayName strings to be more imperative and explanatory.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
eng/pipeline/rolling-internal-validation.gen.yml Adds pipeline definition links and an _info runtime parameter in the generator source.
eng/pipeline/rolling-internal-validation-pipeline.yml Propagates _info parameter and links into the generated official pipeline YAML.
eng/pipeline/rolling-internal-validation-pipeline-unofficial.yml Propagates _info parameter and links into the generated unofficial pipeline YAML.
eng/pipeline/go-docker-rolling-internal.gen.yml Adds pipeline definition links and an _info parameter; updates parameter displayName text in generator source.
eng/pipeline/go-docker-rolling-internal-pipeline.yml Propagates _info parameter/links and updated display names into the official pipeline YAML.
eng/pipeline/go-docker-rolling-internal-pipeline-unofficial.yml Propagates _info parameter/links and updated display names into the unofficial pipeline YAML.
eng/pipeline/go-docker-pr-pipeline.yml Adds standard header and a link to the public pipeline definition.
eng/pipeline/cleanup-acr-images.gen.yml Adds _info parameter/links; reorders/rewrites runtime parameter displayNames; moves derived-from note near extends.
eng/pipeline/cleanup-acr-images-pipeline.yml Propagates _info parameter and updated parameter labels into the official pipeline YAML.
eng/pipeline/cleanup-acr-images-pipeline-unofficial.yml Propagates _info parameter and updated parameter labels into the unofficial pipeline YAML.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread eng/pipeline/cleanup-acr-images.gen.yml
Comment thread eng/pipeline/cleanup-acr-images-pipeline.yml
Comment thread eng/pipeline/cleanup-acr-images-pipeline-unofficial.yml
@dagood dagood marked this pull request as draft July 7, 2026 17:17

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.

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Comment thread eng/pipeline/rolling-internal-validation.gen.yml
Comment thread eng/pipeline/go-docker-rolling-internal.gen.yml
Comment thread eng/pipeline/cleanup-acr-images.gen.yml
@dagood dagood marked this pull request as ready for review July 7, 2026 17:45
@karianna karianna merged commit a3c0cc1 into microsoft/main Jul 8, 2026
29 checks passed
@karianna karianna deleted the dev/dagood/pipe-clarity branch July 8, 2026 09:14
Sign up for free to 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