Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[ci] Remove test environment PATH additions #9712

Merged
merged 1 commit into from
Jan 28, 2025
Merged

[ci] Remove test environment PATH additions #9712

merged 1 commit into from
Jan 28, 2025
+0 −5

Conversation

pjcollins
Copy link
Member

@pjcollins pjcollins commented Jan 27, 2025 •
edited
Loading

A couple of scripts that added various tools to $PATH have been removed.
These were added when we were onboarding to the 1ES pipeline template
and testing new machine pool/image requirements. The Windows images
we're using have received many updates since then, and these workarounds
that updated $PATH are no longer necessary.

@pjcollins
Test PATH removing path updates

Unverified

This user has not yet uploaded their public signing key.
Learn about vigilant mode
Loading
Loading status checks…
676e860
@pjcollins pjcollins changed the title Test PATH removing path updates [ci] Remove test environment PATH additions Jan 27, 2025
@pjcollins pjcollins marked this pull request as ready for review January 27, 2025 21:49
@pjcollins pjcollins requested a review from jonpryor as a code owner January 27, 2025 21:49
@jonathanpeppers jonathanpeppers merged commit f4e405e into main Jan 28, 2025
58 checks passed
@jonathanpeppers jonathanpeppers deleted the dev/pjc/no-prepend branch January 28, 2025 00:59
grendello added a commit that referenced this pull request Jan 28, 2025
@grendello
Merge branch 'main' into dev/grendel/clr-host
26ae8ef 
* main:
  [ci] Remove test environment `$PATH` additions (#9712)
  [Xamarin.Android.Build.Tasks] Fix bug in the Design Time Build (#9711)
  [ci] Set `$(TreatWarningsAsErrors)` = `true` for CI builds. (#9689)
  [XABT] Fix ApplicationAttribute.ManageSpaceActivity manual mapping (#9708)
@github-actions github-actions bot locked and limited conversation to collaborators Feb 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jonathanpeppers jonathanpeppers

@jonpryor jonpryor

Copilot code review Copilot

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@pjcollins @jonathanpeppers