Skip to content

Conversation

@jonthysell
Copy link
Contributor

@jonthysell jonthysell commented May 30, 2025

Description

This PR makes it clearer which property to use with the .ado/templates/run-windows-with-certificates.yml and fixes where it was used wrong in .ado/jobs/integration-test.yml.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

Why

Unbreak CI and prevent future mistakes.

What

Rename deployOption parameter into more understandable moreRunWindowsArgs in .ado/templates/run-windows-with-certificates.yml

Also fixed issue in .ado/jobs/integration-test.yml to unblock CI.

Screenshots

N/A

Testing

Verifying CI

Changelog

Should this change be included in the release notes: no

@jonthysell jonthysell requested review from a team as code owners May 30, 2025 17:58
@jonthysell jonthysell changed the title Rename deployOption paramter into more understandable moreRunWindowsArgs in .ado/templates/run-windows-with-certificates.yml Rename deployOption parameter into more understandable moreRunWindowsArgs in .ado/templates/run-windows-with-certificates.yml May 30, 2025
@jonthysell jonthysell enabled auto-merge (squash) May 30, 2025 17:59
@jonthysell jonthysell merged commit 1290f7d into microsoft:main May 30, 2025
59 checks passed
@jonthysell jonthysell deleted the cirunwindows branch May 30, 2025 21:34
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.

2 participants