Skip to content

Cherry-pick CI/pipeline fixes for rel-1.25.0#28106

Merged
vraspar merged 4 commits intorel-1.25.0from
vraspar/cherry-pick-rel-1.25.0
Apr 17, 2026
Merged

Cherry-pick CI/pipeline fixes for rel-1.25.0#28106
vraspar merged 4 commits intorel-1.25.0from
vraspar/cherry-pick-rel-1.25.0

Conversation

@vraspar
Copy link
Copy Markdown
Contributor

@vraspar vraspar commented Apr 17, 2026

This cherry-picks the following commits for the release:

Commit ID PR Number Commit Title
63dfc33 #28034 fix: idempotent feed setup
58f99ea #28064 [C] fix: ADO CI spuriously failing format check due to picking up NuGet.config
f9c83ae #28084 Fix the python pipeline
cafbd7c #28079 [CI] fix: use internal feed for NPM on ADO

Notes

sanaa-hamel-microsoft and others added 4 commits April 17, 2026 00:38
### Description

Modify ADO pipeline feed setup template to be idempotent.


### Motivation and Context

ADO pipelines are currently stateful, and feed setup requires modifying
files outside of the agent work directories.
Previous jobs may have already modified these files in incompatible
ways, so always override.
…et.config (#28064)

### Description &  Motivation

On ADO CI we modify `/NuGet.config` to add an authenticated feed.
This trips up some format checks, which mistakenly considers unformatted
b/c git reports it as dirty.
Scope to just the CWD and below to avoid picking up `/NuGet.config`.
### Description
<!-- Describe your changes. -->



### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
Use internal NPM feed.

SDL requirements state that we consume everything in 1ES builds from one
internal feed.

---------

Co-authored-by: Erik <erscor@microsoft.com>
@vraspar vraspar enabled auto-merge (squash) April 17, 2026 01:07
@vraspar vraspar merged commit 7a71bc5 into rel-1.25.0 Apr 17, 2026
74 of 77 checks passed
@vraspar vraspar deleted the vraspar/cherry-pick-rel-1.25.0 branch April 17, 2026 13:09
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