Skip to content

Commit

Permalink
CI: Fix triggers for pipeline changes
Browse files Browse the repository at this point in the history
Revert to the earlier triggers which excluded coreclr, installer, or
mono ymls, for the relevant subsets. For example:

```yml
    subset: coreclr
    exclude:
      - eng/pipelines/installer/*
      - eng/pipelines/mono/*
      - eng/pipelines/libraries/*
```

Prompted by dotnet#76057 (comment)
  • Loading branch information
radical committed Oct 4, 2022
1 parent 89c7184 commit 5423ff6
Showing 1 changed file with 14 additions and 9 deletions.
23 changes: 14 additions & 9 deletions eng/pipelines/common/evaluate-default-paths.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,20 +21,12 @@ parameters:
src/tests/Common/wasm-test-runner/*
]
_wasm_pipelines: [
eng/pipelines/*wasm*
eng/pipelines/common/templates/*wasm*
eng/pipelines/common/templates/runtime/wasm*
eng/pipelines/coreclr/*wasm*
eng/pipelines/**/*wasm*
]
# src/workloads is only used in runtime-official builds
# where evaluate-paths is not used
_always_exclude: [
eng/pipelines/common/evaluate-default-paths.yml
eng/pipelines/coreclr/*
eng/pipelines/installer/*
eng/pipelines/libraries/*
eng/pipelines/mono/*
eng/pipelines/runtime*
src/workloads/*
]

Expand All @@ -60,6 +52,9 @@ jobs:
- src/libraries/*
- src/native/libs/*
- src/tests/*
- eng/pipelines/installer/*
- eng/pipelines/mono/*
- eng/pipelines/libraries/*
- ${{ parameters._const_paths._wasm_pipelines }}
- ${{ parameters._const_paths._always_exclude }}

Expand All @@ -85,6 +80,9 @@ jobs:
- src/libraries/*
- src/native/libs/*
- src/tests/*
- eng/pipelines/installer/*
- eng/pipelines/coreclr/*
- eng/pipelines/libraries/*

- subset: libraries
exclude:
Expand All @@ -99,6 +97,9 @@ jobs:
- src/coreclr/*
- src/tests/*
- src/native/eventpipe/*
- eng/pipelines/coreclr/*
- eng/pipelines/mono/*
- eng/pipelines/installer/*
- ${{ parameters._const_paths._wasm_pipelines }}
- ${{ parameters._const_paths._always_exclude }}

Expand Down Expand Up @@ -127,6 +128,9 @@ jobs:
- src/libraries/*
- src/tests/*
- src/native/eventpipe/*
- eng/pipelines/coreclr/*
- eng/pipelines/mono/*
- eng/pipelines/libraries/*
- ${{ parameters._const_paths._wasm_pipelines }}
- ${{ parameters._const_paths._always_exclude }}

Expand Down Expand Up @@ -237,6 +241,7 @@ jobs:
# anything other than mono, or wasm specific paths
- subset: non_mono_and_wasm
exclude:
- eng/pipelines/mono/*
- ${{ parameters._const_paths._wasm_specific_only }}
- ${{ parameters._const_paths._wasm_pipelines }}
- ${{ parameters._const_paths._always_exclude }}
Expand Down

0 comments on commit 5423ff6

Please sign in to comment.