Skip to content

Conversation

@azure-sdk
Copy link
Collaborator

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#9635 See eng/common workflow

scbedd and others added 2 commits January 15, 2025 18:44
… are NOT counted as 'includedForValidation: true' if they also exist in the original 'these packages changed' set
@azure-sdk azure-sdk requested a review from a team as a code owner January 15, 2025 20:21
@azure-sdk azure-sdk requested a review from scbedd January 15, 2025 20:21
@azure-sdk azure-sdk added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Jan 15, 2025
@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Jan 15, 2025

Next Steps to Merge

⌛ Please wait. Next steps to merge this PR are being evaluated by automation. ⌛

@openapi-pipeline-app
Copy link

PR validation pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

@scbedd scbedd merged commit 7e08d30 into main Jan 15, 2025
33 of 35 checks passed
@scbedd scbedd deleted the sync-eng/common-handle-additionalvalidation-overriding-includedForvalidation-9635 branch January 15, 2025 21:36
pjpatel12 pushed a commit to pjpatel12/azure-rest-api-specs that referenced this pull request Apr 29, 2025
* ensure that packages that come back from AdditionalValidationPackages are NOT counted as 'includedForValidation: true' if they also exist in the original 'these packages changed' set

Co-authored-by: Ben Broderick Phillips <[email protected]>

---------

Co-authored-by: Scott Beddall <[email protected]>
Co-authored-by: Scott Beddall <[email protected]>
Co-authored-by: Ben Broderick Phillips <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants