Skip to content

Conversation

necusjz
Copy link
Member

@necusjz necusjz commented Feb 19, 2025

switch via dropdown menu.

generic first:
image

patch first:
image

@necusjz necusjz requested a review from kairu-ms February 19, 2025 00:33
@necusjz necusjz self-assigned this Feb 19, 2025
kairu-ms
kairu-ms previously approved these changes Jun 4, 2025
@@ -744,10 +744,73 @@ def find_sub_schema(cls, schema, idx):

return None

@classmethod
def trim_schema_by_idx(cls, schema, idx): # only keep identity and its parents
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please handle the error case when idx is invalid

Copy link
Member Author

@necusjz necusjz Jun 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idx can be null.

@necusjz necusjz merged commit 5c05310 into Azure:dev Jun 20, 2025
3 checks passed
@necusjz necusjz mentioned this pull request Jul 9, 2025
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