Commit c3afba3
Skip localization PR creation on file-change CI trigger (#40501)
* Skip localization PR creation on file-change CI trigger
The nightly localization pipeline runs both on schedule and when
localization/strings/en-US/Resources.resw is updated. The CI-triggered
run produces a no-op Touchdown pass that still creates a PR, consuming
build resources without updating any strings.
Gate the Create-PR step on Build.Reason in (Schedule, Manual) so the PR
is only created for the nightly schedule (and manual runs), not when
the pipeline is triggered by a push to Resources.resw.
Follow-up to #40499
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Also gate PipAuthenticate on Schedule/Manual
The pip auth step only exists to support create-change.py; if we're not
creating a PR there's no reason to authenticate to the pip feed.
Keep TouchdownBuildTask running on file-change CI so new source strings
are still submitted to the translation service.
Addresses review feedback from @OneBlue.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
---------
Co-authored-by: Ben Hillis <benhill@ntdev.microsoft.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>1 parent 2b3fc85 commit c3afba3
1 file changed
Lines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
43 | 43 | | |
44 | 44 | | |
45 | 45 | | |
| 46 | + | |
46 | 47 | | |
47 | 48 | | |
48 | 49 | | |
| |||
51 | 52 | | |
52 | 53 | | |
53 | 54 | | |
| 55 | + | |
54 | 56 | | |
55 | 57 | | |
56 | 58 | | |
| |||
0 commit comments