Skip to content

Conversation

wtgodbe
Copy link
Member

@wtgodbe wtgodbe commented May 20, 2025

Matches what's in ci-public and quarantined-test

This pull request updates the .azure/pipelines/components-e2e-tests.yml file to exclude specific paths from triggering the pipeline.

Pipeline configuration updates:

  • Added a paths.exclude section to prevent changes in the following paths from triggering the pipeline:
    • .devcontainer/*
    • .github/*
    • .vscode/*
    • docs/*
    • **/*.md
    • LICENSE.TXT
    • THIRD-PARTY-NOTICES.TXT

@wtgodbe wtgodbe requested a review from Copilot May 20, 2025 21:53
@wtgodbe wtgodbe requested a review from a team as a code owner May 20, 2025 21:53
@github-actions github-actions bot added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label May 20, 2025
Copy link
Contributor

@Copilot Copilot AI left a 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 updates the components-e2e pipeline to avoid running on non-code changes by excluding specific file paths from PR triggers.

  • Add paths.exclude under the pr trigger
  • Exclude config directories, docs, markdown files, and license notices
Comments suppressed due to low confidence (2)

.azure/pipelines/components-e2e-tests.yml:22

  • Glob patterns containing '*' should be quoted to avoid YAML alias interpretation (e.g., change .devcontainer/* to ".devcontainer/*" ).
- .devcontainer/*

.azure/pipelines/components-e2e-tests.yml:20

  • [nitpick] Consider extracting this path exclusion list into a shared template or variable to reduce duplication across multiple pipeline configs.
paths:

@wtgodbe wtgodbe merged commit 4d10511 into main May 20, 2025
4 of 28 checks passed
@wtgodbe wtgodbe deleted the wtgodbe/ce2e branch May 20, 2025 21:58
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0-preview5 milestone May 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant