Skip to content

Conversation

@konrad-jamrozik
Copy link
Contributor

@konrad-jamrozik konrad-jamrozik commented Jan 31, 2023

This PR fixes a bug introduced by:

I.e. by converting path
/sdk/ml/azure-ai-ml/azure/ai/ml/_schema/job
into
/sdk/ml/azure-ai-ml/azure/ai/ml/_schema/job/
I made it stop matching [1] against
sdk/ml/azure-ai-ml/azure/ai/ml/_schema/job_resource_configuration.py

This fixes it.

[1] Strictly speaking the path may have not been matching in the first place, per .gitignore doc and CODEOWNERS doc. However, it seems to me what this PR proposes is in fact the intended behavior.

@konrad-jamrozik konrad-jamrozik added the Central-EngSys This issue is owned by the Engineering System team. label Jan 31, 2023
@konrad-jamrozik konrad-jamrozik requested review from a team, benbp and weshaggard January 31, 2023 06:48
@konrad-jamrozik konrad-jamrozik self-assigned this Jan 31, 2023
@konrad-jamrozik konrad-jamrozik changed the title Add path erroneously removed from CODEOWNERS. Add path erroneously removed from CODEOWNERS Jan 31, 2023
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants